Skip to content

Add requestHeaderBlacklist and requestHeaderWhitelist#81

Merged
tdipisa merged 2 commits intogeosolutions-it:masterfrom
f-necas:white-black-header-list
Mar 30, 2026
Merged

Add requestHeaderBlacklist and requestHeaderWhitelist#81
tdipisa merged 2 commits intogeosolutions-it:masterfrom
f-necas:white-black-header-list

Conversation

@f-necas
Copy link
Copy Markdown
Contributor

@f-necas f-necas commented Mar 16, 2026

This PR allows to set B&W whitelists to avoid sending all headers to requested Host.

Blacklist takes precedence over whitelist. If a header is set in both list, it will be removed.

Compiler plugin source & target have been set to level 8 in order to use java streams.

Closes #80

@tdipisa tdipisa added this to the 1.6.0 milestone Mar 17, 2026
@tdipisa tdipisa requested a review from axl8713 March 17, 2026 14:35
Copy link
Copy Markdown
Member

@tdipisa tdipisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @f-necas thank you for this PR. I notice a failing check in the build.

@f-necas
Copy link
Copy Markdown
Contributor Author

f-necas commented Mar 17, 2026

@tdipisa Damn, it's because I use Java streams. Is there a reason why we don't use Maven Compiler in version 8 ?

It was completely intentional, can the workflow be upgraded also ? If not, I'll remove the Java stream.

@f-necas f-necas requested a review from tdipisa March 18, 2026 15:27
@offtherailz
Copy link
Copy Markdown
Member

I don't think there is a reason in particular to keep source and target to 1.7.
Last time has been updated in 2021 . On geostore is 11, on MapStore 8.
So I think 8 can be fine.

@f-necas
Copy link
Copy Markdown
Contributor Author

f-necas commented Mar 25, 2026

@offtherailz thanks for the precision. I updated the pom.xml to use 8. Just need an approval to check if CI is ok.

@tdipisa tdipisa requested a review from offtherailz March 30, 2026 09:23
@tdipisa tdipisa self-assigned this Mar 30, 2026
@tdipisa tdipisa merged commit 2bb7397 into geosolutions-it:master Mar 30, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a headerWhitelist config property

4 participants