-
Notifications
You must be signed in to change notification settings - Fork 611
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
Fix PasswordDigestExt algorithm #265
base: next
Are you sure you want to change the base?
Conversation
This fixes the PasswordDigestExt algorithm to match the definition described in the ws-security documentation (using the documentation as expectation). Documentation: Section about "Username" and it's configurable fields https://www.soapui.org/soapui-projects/ws-security.html Tested against Amadeus Web Services. Credential was always being reported invalid prior to this fix.
This fixes the PasswordDigestExt algorithm to match the definition described in the ws-security documentation (using the documentation as expectation). Documentation: Section about "Username" and it's configurable fields https://www.soapui.org/soapui-projects/ws-security.html Tested against Amadeus Web Services. Credential was always being reported invalid prior to this fix.
…apui into FixPasswordDigestExt
thanks! - perhaps we need to add a setting for this as it could break usage for those that depend on unencrypted passwords!? |
Technically the current version is encrypting the password but it is also performing an additional base64 encoding on top of the sha-1 encryption of the clear text password. Without looking at the source code there would be no way to get a server side to match this nonstandard approach. Here is what the current oasis standard is (Password Digest): Here is what the SoapUI documentation suggests the nonstandard interop approach is (Password Digest Ext): Here is what SoapUI actually does for Password Digest Ext: If you are concerned about those that were using the Password Digest Ext successfully (could be they looked at the source code and adjusted their server side to match) there could be a Password Digest SoapUI version I suppose? |
Any update on this? I am also trying to use soapui for Amadeus webservices and am experiencing the same issue. |
Hi @unalyildirim - thanks for reaching out - unfortunately I don't have write access to this repo - you'll have to ask someone from the maintainer/smartbear team.. good luck! |
This fixes the PasswordDigestExt algorithm to match the definition described in the ws-security documentation (using the documentation as expectation).
Documentation:
Section about "Username" and it's configurable fields
https://www.soapui.org/soapui-projects/ws-security.html
Tested against Amadeus Web Services. Credential was always being reported invalid prior to this fix.