Page MenuHomePhabricator
Differential D9187

[docs] Use 127.0.0.1 in db_config.json instead of localhost
ClosedPublic
Actions

Authored by ashoat on Sep 13 2023, 10:04 AM.
Tags
None
Referenced Files
F3388900: D9187.id31081.diff
Fri, Nov 29, 4:33 PM
F3388782: D9187.id31082.diff
Fri, Nov 29, 3:57 PM
F3388700: D9187.id31074.diff
Fri, Nov 29, 3:38 PM
F3388339: D9187.diff
Fri, Nov 29, 1:47 PM
Unknown Object (File)
Oct 27 2024, 3:13 PM
Unknown Object (File)
Oct 27 2024, 3:13 PM
Unknown Object (File)
Oct 27 2024, 3:13 PM
Unknown Object (File)
Oct 27 2024, 3:13 PM
View All 89 Files
Subscribers
None

Details

Reviewers
atul
varun
tomek
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rCOMMf906621ba466: [docs] Use 127.0.0.1 in db_config.json instead of localhost
Summary

For some reason, after coming back from Burning Man, I am unable to connect to my local MariaDB from Node.js with localhost. Once I switched to 127.0.0.1, everything works fine.

I'm not sure if this is due to some macOS update or something else, but I figure this docs change will help other devs avoid this issue.

Test Plan

I confirmed 127.0.0.1 worked for me when localhost didn't

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat created this revision.Sep 13 2023, 10:04 AM
Harbormaster completed remote builds in B22604: Diff 31074.Sep 13 2023, 10:29 AM
ashoat requested review of this revision.Sep 13 2023, 10:29 AM
atul accepted this revision.Sep 13 2023, 10:32 AM
This revision is now accepted and ready to land.Sep 13 2023, 10:32 AM
atul added a comment.Sep 13 2023, 10:33 AM

Is this config auto generated by nix somehow? Should mariadb-up-mac.nix be updated as well?

ashoat updated this revision to Diff 31081.Sep 13 2023, 10:57 AM

Also update mariadb-up-mac.nix

Owners added a reviewer: Restricted Owners Package.Sep 13 2023, 10:57 AM
This revision now requires review to proceed.Sep 13 2023, 10:57 AM
ashoat added a comment.Sep 13 2023, 10:57 AM

Good call

ashoat removed 1 blocking reviewer(s): Restricted Owners Package.Sep 13 2023, 10:59 AM
This revision is now accepted and ready to land.Sep 13 2023, 10:59 AM
This revision was landed with ongoing or failed builds.Sep 13 2023, 10:59 AM
Closed by commit rCOMMf906621ba466: [docs] Use 127.0.0.1 in db_config.json instead of localhost (authored by ashoat). · Explain Why
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMMf906621ba466: [docs] Use 127.0.0.1 in db_config.json instead of localhost.
Harbormaster completed remote builds in B22609: Diff 31081.Sep 13 2023, 11:15 AM

Revision Contents
Changeset List

PathSize
docs/
2 lines
nix/
2 lines

Diff 31082

View Options

docs/dev_environment.md

Loading...
View Options

nix/mariadb-up-mac.nix

Loading...