Skip to content

Conversation

@moresandeep
Copy link
Contributor

What changes were proposed in this pull request?

Fix an issue where group headers were not added to POST and PUT requests

How was this patch tested?

This patch was tested locally with POST and GET requests.

@moresandeep moresandeep self-assigned this Sep 6, 2024
@moresandeep moresandeep closed this Sep 6, 2024
@moresandeep moresandeep reopened this Sep 15, 2025
@smolnar82
Copy link
Contributor

@moresandeep - I think is still a valid issue to fix. Would you mind taking a review, update/rebase your change and submit a new patch? Thanks!

@pzampino
Copy link
Contributor

pzampino commented Jan 7, 2026

How are these changes specific to POST and PUT requests?

@smolnar82
Copy link
Contributor

Looking closer again, I found this one irrelevant. The reason is, that the submitted code here assumes that #747 is merged, which isn't the case.
Thus, this PR can be closed as we cover all REST methods in our dispatch properly in terms of group headers addition.

@moresandeep
Copy link
Contributor Author

I closed the JIRA a while ago and marked it as not an issue https://issues.apache.org/jira/browse/KNOX-3062
We don't need this.

@moresandeep moresandeep closed this Jan 8, 2026
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.

3 participants