Skip to content

gs-stable CORS policy issue on QA #11234#11238

Merged
offtherailz merged 1 commit intogeosolutions-it:masterfrom
rowheat02:fix/gs-stableauthkeyavoid
Jun 23, 2025
Merged

gs-stable CORS policy issue on QA #11234#11238
offtherailz merged 1 commit intogeosolutions-it:masterfrom
rowheat02:fix/gs-stableauthkeyavoid

Conversation

@rowheat02
Copy link
Copy Markdown
Contributor

Description

As sending the authkey(configured in localConfig) throws an error on the qa server, as mentioned here #11234. So, reverting the change by removing the authkey rule from authenticationRules.

**Please check if the PR fulfills these requirements **

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

fixes #11234

What is the current behavior?

Error from services(gs-stable) when authkey is sent.

What is the new behavior?

No errror as no authkey is being be sent.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

@rowheat02 rowheat02 added this to the 2025.01.01 milestone Jun 23, 2025
@rowheat02 rowheat02 self-assigned this Jun 23, 2025
@offtherailz offtherailz added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Jun 23, 2025
@offtherailz offtherailz enabled auto-merge (squash) June 23, 2025 16:30
@offtherailz offtherailz merged commit ec70ab0 into geosolutions-it:master Jun 23, 2025
6 checks passed
@offtherailz
Copy link
Copy Markdown
Member

No tests required on Master branch here @ElenaGallo
The test will be required on QA, when backported.

@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

gs-stable CORS policy issue on QA

3 participants