Fix #5678

[dplewis]

No description provided.

[codecov]

Codecov Report

Merging #5681 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5681      +/-   ##
==========================================
- Coverage   94.19%   94.16%   -0.03%     
==========================================
  Files         129      129              
  Lines        9248     9240       -8     
==========================================
- Hits         8711     8701      -10     
- Misses        537      539       +2

Impacted Files	Coverage Δ
src/Routers/FeaturesRouter.js	100% <ø> (ø)	⬆️
...dapters/Storage/Postgres/PostgresStorageAdapter.js	96.81% <ø> (-0.01%)	⬇️
src/Adapters/Storage/Mongo/MongoStorageAdapter.js	92.45% <100%> (-0.08%)	⬇️
src/RestWrite.js	93.15% <0%> (-0.35%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 089b809...7af4944. Read the comment docs.

[acinader]

while this fix seems fine, my question at this point is: what do we get out of including the mysql in server info? Thinking about why this is an issue, I'm pretty convinced that a production deployment should never allow the parse-server to ask this question as the parse-server doesn't need and shouldn't have the rights.

cc: @oallouch

[dplewis]

@acinader @oallouch I removed this feature as there a lot of use cases where this wouldn't work. Such as ensuring users have admin access, which isn't always the case.

[oallouch]

I understand. Hope I can help with something else.