Skip to content

Params wmts#11824

Merged
offtherailz merged 2 commits intogeosolutions-it:masterfrom
nordiq-group:params_wmts
Dec 17, 2025
Merged

Params wmts#11824
offtherailz merged 2 commits intogeosolutions-it:masterfrom
nordiq-group:params_wmts

Conversation

@tha-nordiq
Copy link
Copy Markdown
Contributor

Description

layer.params are ignored for wmts layers. A specific wmts provider requires 'username'- and 'password'-url parameters.

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:

Issue

#10999

What is the current behavior?
options.params are dropped

What is the new behavior?
options.params are build into the url

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

@tha-nordiq tha-nordiq mentioned this pull request Dec 17, 2025
1 task
@tdipisa tdipisa linked an issue Dec 17, 2025 that may be closed by this pull request
1 task
@tdipisa tdipisa added this to the 2025.02.01 milestone Dec 17, 2025
@tdipisa tdipisa requested a review from offtherailz December 17, 2025 14:03
@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Dec 17, 2025

@tha-nordiq thank your very much for your contribution.

@offtherailz
Copy link
Copy Markdown
Member

@tha-nordiq nice catch. Thank you very much for your fix 👍

@offtherailz offtherailz merged commit 603f9a0 into geosolutions-it:master Dec 17, 2025
20 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 17, 2025
(cherry picked from commit 603f9a0)
@github-actions
Copy link
Copy Markdown
Contributor

Successfully created backport PR for 2025.02.xx:

mahesh-wor pushed a commit to mahesh-wor/MapStore2 that referenced this pull request Dec 17, 2025
@offtherailz
Copy link
Copy Markdown
Member

no checks necessary, set to accepted

offtherailz pushed a commit that referenced this pull request Dec 17, 2025
(cherry picked from commit 603f9a0)

Co-authored-by: Thomas Hallundbæk <72447095+tha-nordiq@users.noreply.github.com>
allyoucanmap pushed a commit that referenced this pull request Mar 13, 2026
(cherry picked from commit 603f9a0)

Co-authored-by: Thomas Hallundbæk <72447095+tha-nordiq@users.noreply.github.com>
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.

wmts url-params are removed

3 participants