Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dynamic httpconfiguration properties #176

Merged
merged 5 commits into from
Jan 30, 2024

Conversation

olamy
Copy link
Contributor

@olamy olamy commented Jan 29, 2024

  • Add feature to be able to configuration dynamically httpConfiguration properties. With such format
        <property name="httpConfigurationProperties">
          relativeRedirectAllowed false
        </property>

Short description of what this resolves:

Changes proposed in this pull request:

Fixes: #

olamy added 2 commits January 29, 2024 12:07
… properties.

With such format

        <property name="httpConfigurationProperties">
          relativeRedirectAllowed false
        </property>

Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
@olamy olamy force-pushed the dynamic-httpconfiguration-properties branch from aabe208 to 35615af Compare January 29, 2024 02:08
@olamy olamy requested a review from joakime January 29, 2024 11:17
@olamy olamy assigned olamy and unassigned joakime Jan 29, 2024
Copy link
Contributor

@joakime joakime left a comment

Choose a reason for hiding this comment

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

Space delimiting is broken for configuration properties.
Pick something else.

olamy added 3 commits January 30, 2024 13:29
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
@olamy olamy requested a review from joakime January 30, 2024 03:38
@olamy olamy merged commit 7237090 into arquillian:master Jan 30, 2024
6 checks passed
@olamy olamy deleted the dynamic-httpconfiguration-properties branch January 30, 2024 20:57
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.

2 participants