Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Log a warning in case of a unknown options #8938

Closed
3 tasks done
EhsanParsania opened this issue Feb 23, 2024 · 7 comments · Fixed by #8987
Closed
3 tasks done

Log a warning in case of a unknown options #8938

EhsanParsania opened this issue Feb 23, 2024 · 7 comments · Fixed by #8987
Labels
bounty:$20 Bounty applies for fixing this issue (Parse Bounty Program) state:released Released as stable version state:released-alpha Released as alpha version state:released-beta Released as beta version type:feature New feature or improvement of existing feature

Comments

@EhsanParsania
Copy link
Member

New Feature / Enhancement Checklist

Current Limitation

When we pass a wrong option in Parse config it doesn't make a warning,

Feature / Enhancement Description

There should be a validation or list of giving options to prevent such mistakes with a warning.

Example Use Case

for instance in the Parse config I had a typo like:
masterKeyIPs which was the correct form was masterKeyIps and it took time to find out why I got error.
@parse-github-assistant Parse GitHub Assistant
Copy link

Thanks for opening this issue!

  • 🎉 We are excited about your ideas for improvement!

@EhsanParsania EhsanParsania mentioned this issue Feb 23, 2024
Closed
4 tasks
@mtrezza mtrezza added type:feature New feature or improvement of existing feature bounty:$20 Bounty applies for fixing this issue (Parse Bounty Program) labels Feb 23, 2024
@vivekjoshi556
Copy link
Contributor

vivekjoshi556 commented Mar 2, 2024 •
edited
Loading

@mtrezza Can I try this issue?

@mtrezza
Copy link
Member

mtrezza commented Mar 2, 2024

Sure, please go ahead

@vivekjoshi556 vivekjoshi556 mentioned this issue Mar 4, 2024
Merged
4 tasks
@mtrezza
Copy link
Member

mtrezza commented Mar 15, 2024

Related: #7069

@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 7.1.0-alpha.5

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Apr 7, 2024
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 7.1.0-beta.1

@parseplatformorg parseplatformorg added the state:released-beta Released as beta version label Jun 30, 2024
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 7.1.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bounty:$20 Bounty applies for fixing this issue (Parse Bounty Program) state:released Released as stable version state:released-alpha Released as alpha version state:released-beta Released as beta version type:feature New feature or improvement of existing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Prevent Parse Server start in case of unknown option in server configuration
4 participants
@mtrezza @vivekjoshi556 @EhsanParsania @parseplatformorg