Skip to content

workaround for #67: decode curly brackets again#68

Open
thijsbrentjens wants to merge 1 commit intomasterfrom
fix-issue-67-url-encoding-metdata
Open

workaround for #67: decode curly brackets again#68
thijsbrentjens wants to merge 1 commit intomasterfrom
fix-issue-67-url-encoding-metdata

Conversation

@thijsbrentjens
Copy link
Member

No description provided.

@jonherrmann
Copy link
Collaborator

Looks good, that should work for both encodings.

@thijsbrentjens
Copy link
Member Author

Not sure if this is a workaround suitable for the webapp. I get this error on our test instance:

java.net.URISyntaxException: Illegal character in query at index 53: https://...../csw?getxml={33F9A1CB-6520-4953-8794-88BCB8D637B5}error at line: 21

referring to the curly bracket that has been decoded..

@thijsbrentjens
Copy link
Member Author

Sorry, my previous comment is not correct. But there is still an issue. Will come back to it.

@jonherrmann
Copy link
Collaborator

jonherrmann commented Aug 19, 2016

Then we could give the RestUtils function a try:

// put this into a groovy test step, change HTTP "TestStepName", call after properties are set and before HTTP test step is called.
import com.eviware.soapui.impl.rest.support.RestUtils;
import de.interactive_instruments.xtf.*;
SOAPUI_I.init(log,context,testRunner);

ProjectHelper ph = new ProjectHelper()
def targetTestStep = ph.getTestStep("TestStepName")
RestUtils.extractParams(targetTestStep.getHttpRequest().getEndpoint(),
                targetTestStep.getHttpRequest().getParams(), true)

@thijsbrentjens
Copy link
Member Author

Hmm, I can't get it working now. See #67 for more comments.
It seems that SoapUI is quite aggressive in encoding the request parameters again. Even if they are already encoded properly in the URL to test.

( @jonherrmann: maybe you already were aware of this, but maybe it helps in finding a solution)

@jonherrmann
Copy link
Collaborator

To give a short update here: i made tests with some new SoapUI 5.2.1 plugin features and it looks like this will solve the problem.

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.

2 participants