nginx custom error message - Githubissues

microsoft / oxa-tools

Open edX on Azure Tools

MIT License

36 stars 103 forks source link

nginx custom error message #387

Closed prabhanshu closed 5 years ago

prabhanshu commented 5 years ago

What does this PR do? Please provide some context

It will override the default NGINX error message

Where should the reviewer start?

How can this be manually tested? (brief repro steps and corpnet-URL with change)

What are the relevant TFS items? (list id numbers)

Definition of done:

[ ] Title of the pull request is clear and informative
[ ] Add pull request hyperlink to relevant TFS items
[ ] For large or complex change: schedule an in-person review session
[ ] This change has appropriate test coverage
[ ] Get at least two approvals

Reminders DURING merge

If you're merging from a short-term (feature) branch into a long-term branch (like dev, release, or master) then "Squash and merge" to keep our history clean.
If merging from two longterm branches (like cherry picks from upstream, dev to release, etc) then "Create merge commit" to preserve individual commits.

prabhanshu commented 5 years ago

Based on the @eltoncarr feedback related to Nginx error message in edx-configuration PR (https://github.com/Microsoft/edx-configuration/pull/165) I have added the configuration which will override the default message