Page MenuHomePhabricator
Differential D16901

Respect user/pass flags to bin/storage for direct DatabaseRef-based queries
ClosedPublic
Actions

Authored by epriestley on Nov 19 2016, 4:38 PM.
Tags
None
Referenced Files
F19079676: D16901.id40687.diff
Mon, Dec 1, 9:37 PM
F18876414: D16901.id.diff
Thu, Nov 6, 12:21 AM
F18855046: D16901.diff
Nov 1 2025, 2:29 AM
F18775844: D16901.id.diff
Oct 10 2025, 8:23 PM
F18736688: D16901.id.diff
Oct 1 2025, 7:12 AM
F18700950: D16901.id40687.diff
Sep 27 2025, 4:26 PM
F18618300: D16901.diff
Sep 14 2025, 11:41 PM
F18618295: D16901.diff
Sep 14 2025, 11:40 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11893: storage upgrade claims it lacks permissions, but is running as root mysql user
Commits
rP74a0e5cd7930: (stable) Respect user/pass flags to bin/storage for direct DatabaseRef-based…
rP8fbb411a9a72: Respect user/pass flags to bin/storage for direct DatabaseRef-based queries
Summary

Ref T11893. Previously, we excuted all bin/storage queries through StorageManagementAPI objects.

After D16848, we execute some queries through PhabricatorDatabaseRef. However, the refs we use weren't getting passed the -u / -p flags correctly, for specifying alternate administrative credentials.

Test Plan
  • Created a second account (trunk).
  • Ran bin/storage -u trunk.
  • Made libphutil throw when not connecting as that user.
  • Before patch: some connections incorrectly used the default user, "root".
  • After patch: all connections correctly used the configured user, "trunk".

Diff Detail

Repository
rP Phabricator
Branch
user1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 14580
Build 19026: Run Core Tests
Build 19025: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 40687.Nov 19 2016, 4:38 PM
epriestley retitled this revision from to Respect user/pass flags to bin/storage for direct DatbaseRef-based queries.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.
epriestley mentioned this in T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.Nov 19 2016, 4:38 PM
chad accepted this revision.Nov 19 2016, 4:38 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 19 2016, 4:38 PM
epriestley retitled this revision from Respect user/pass flags to bin/storage for direct DatbaseRef-based queries to Respect user/pass flags to bin/storage for direct DatabaseRef-based queries.Nov 19 2016, 4:38 PM
epriestley edited edge metadata.
Closed by commit rP8fbb411a9a72: Respect user/pass flags to bin/storage for direct DatabaseRef-based queries (authored by epriestley, committed by epriestley). · Explain WhyNov 19 2016, 4:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
scripts/
sql/
3 lines

Diff 40687

View Options

scripts/sql/manage_storage.php

Loading...
Phabricator · Built by Phacility