Skip to content

Conversation

@free-ice
Copy link

This adds the ability to control wss4j's InclusivePrefixes settings.

This sets the defaults to match wss4j defaults.

#75 should be closed since it would have changed the default behavour of spring-ws.

Tests verify changes and I feel some of the test are overkill but I wanted to ensure that there was no reason not to merge this PR.

I have tested these changes in the 2.4.7 branch of spring-ws, and will be upgrading to this version once the change is merged.

Thanks!

@pivotal-cla
Copy link

@free-ice Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@free-ice Thank you for signing the Contributor License Agreement!

@free-ice
Copy link
Author

free-ice commented Oct 1, 2021

@gregturn Any feedback would be welcome.

@free-ice
Copy link
Author

I'll be moving to a new client between Feb 2022 and April 2022. If there are any required changes now is the time to make comments since after that time I will not have a way to manually test any requested changes in a meaningful way.

@free2create
Copy link

@gregturn @rmagrin would you be able to assign a reviewer to this? I opened this about a year ago and would love to do what I can to see this merged.

Thanks

@ghost
Copy link

ghost commented Aug 23, 2022

I commented on this solution 4 years ago and asked for a release. I don't understand why this can't finally be solved. Compatibility has broken down between wss4j 1.5.x and 2.0.x anyway.

@free2create
Copy link

free2create commented Aug 23, 2022

Greg seems to be the only/main contributor to spring-ws and it seems he is just trying to keep the lights on. Doing OSS can really suck if the projects you work on are on your own time AND you don't use the project yourself. That's not to say its not frustrating, since I don't know if I am not doing something I should be to make the enhancement acceptable.

The Wss4J team also seems to be down to one developer , but Colm(Wss4j) provides good feedback after gentle prodding, so at least the core framework is on better footing.

@free2create
Copy link

@gregturn
I see you made a comment on a similar PR #1239 earlier today, can you review this PR please?

@ghost
Copy link

ghost commented Nov 16, 2022

Yes #1239 is similar, so please give this PR a chance and merge it into release 👍

@free2create
Copy link

@gregturn I no longer have access to the free-ice account since two-factor is enabled. Can you close this PR and I'll create another PR to resolve the merge conflicts. Thanks!

@free2create
Copy link

Please close, This PR is super ceded by PR #1300

@gregturn gregturn closed this Jan 25, 2023
gregturn pushed a commit that referenced this pull request Jan 25, 2023
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.

6 participants