Fix COG and FlatGeobuf layers support params for protected storage#12179
Merged
allyoucanmap merged 2 commits intogeosolutions-it:masterfrom Mar 31, 2026
Conversation
fix cog catalog sugin security params new function setSecurityParams restore config unit test for setSecurityParams securel params in 3d layers fix test SecurityUtils
allyoucanmap
requested changes
Mar 30, 2026
stefanocudini
force-pushed
the
fix-12162-cogs-flatgeobuf-from-protected-storage
branch
from
8e0d3ce to
22e6bba
Compare
allyoucanmap
approved these changes
Mar 31, 2026
Contributor
|
@ElenaGallo please test this fix on dev and let us know when we can backport, thanks |
Member
|
Successfully created backport PR for |
tdipisa
pushed a commit
that referenced
this pull request
Apr 10, 2026
dsuren1
added a commit
to anup39/MapStore2
that referenced
this pull request
Apr 16, 2026
…log_refactor_genode_int_11930 * commit 'de55770d13eb703b5b6cfe31ae55f38fea0b9fcf': Fix geosolutions-it#12146: ArcGIS support for FeatureService (geosolutions-it#12235) geosolutions-it#12094: Fix dropdown menu overflow when adding links to text widget in TextEditor (geosolutions-it#12260) geosolutions-it#12207: Fix - ScaleBox plugin - Custom scales not reported on map load (geosolutions-it#12212) fix a few danish translations (geosolutions-it#12230) geosolutions-it#12223: fix rule manager field filtering with checkbox is not working properly (geosolutions-it#12224) geosolutions-it#12218: fix pagination in rules manager layers autocomplete does not work (geosolutions-it#12219) Fix geosolutions-it#12213 - Fixed startup script for java new versions (geosolutions-it#12214) Update Changelog for version 2025.02.03 (geosolutions-it#12211) Fix pre-release gh action workflow for CLA (geosolutions-it#12205) Update Changelog for version 2026.01.00 (geosolutions-it#12202) fix geosolutions-it#12162 flatgeobuf params by getRequestConfigurationByUrl (geosolutions-it#12200) Fix geosolutions-it#12162 COG and FlatGeobuf layers support params for protected storage (geosolutions-it#12179) Fix doc of MapImport plugin (geosolutions-it#12197) Fix geosolutions-it#12044: fix wfs layer hidden in 3D + scale arrow not rendering (geosolutions-it#12155)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix #12162
Description
In short, in cases of datasets using custom rules in the requestsConfigurationRules section of localConfig.json, the URL parameters for resource addresses corresponding to COG and FlatGeobuf files were not being passed. See the issue for an example of rules to use for testing.
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
Fix #12162
What is the current behavior?
#
What is the new behavior?
A new method
setSecurityParamshas been added to avoid code replication that automatically appends parameters to the URL when necessary. In cases where no rules exist, the original URL is usedBreaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information
See the linked issue for bug reproducing details.
Geonode-client porting is required