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
Unknown Object (File)
Tue, Sep 3, 1:34 AM
Unknown Object (File)
Tue, Aug 27, 8:21 AM
Unknown Object (File)
Mon, Aug 26, 7:03 AM
Unknown Object (File)
Sun, Aug 25, 4:52 PM
Unknown Object (File)
Sun, Aug 25, 1:18 AM
Unknown Object (File)
Thu, Aug 8, 8:43 PM
Unknown Object (File)
Thu, Aug 8, 8:43 PM
Unknown Object (File)
Thu, Aug 8, 8:43 PM
View All 71 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
Branch
ashoat/docs
Lint
No Lint Coverage
Unit
No Test Coverage

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

Diff 31074

View Options

docs/dev_environment.md

Loading...