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

error response when modelNumber includes a / #2

Open
dingman opened this issue Sep 6, 2017 · 9 comments
Open

error response when modelNumber includes a / #2

dingman opened this issue Sep 6, 2017 · 9 comments
Assignees
Labels

Comments

@dingman
Copy link

dingman commented Sep 6, 2017

When I pass in modelNumber=MUF-128BA/AM

it throws an error. the forward / is not very happy.

I tried URL encoding the value, even double URL encoding, but to not avail, it still returns an error.

Is there something possibly server-side in the configuration that is preventing even a URL encoded string from being parsed correctly?

@troymccabe troymccabe self-assigned this Sep 6, 2017
@troymccabe
Copy link
Contributor

troymccabe commented Sep 6, 2017

Looking into this. Running this through the querybuilder here reproduces the issue with the service, and I see the same thing with you & encoding.

Might have to replace forward slashes with a wildcard. @ecaron Any insight on forward slashes in string matches?

@dingman
Copy link
Author

dingman commented Sep 6, 2017

Thanks for jumping in so quickly @troymccabe

I also noticed using a ( or ) caused the same issue.

@Flet
Copy link

Flet commented Sep 7, 2017

FYI I sent a note to our API folks to get more eyes on this.

@troymccabe
Copy link
Contributor

👍 thanks @Flet. Let me know if there's a way we can handle it on the client end

@troymccabe troymccabe added the bug label Sep 7, 2017
@troymccabe
Copy link
Contributor

@Flet Anything from the API team? I haven't found a work-around (other than replacing these characters with a wildcard) on the client-side.

@dingman
Copy link
Author

dingman commented Sep 13, 2017

I also noticed when using the wildcard first, causes the same error.
For example, modelNumber=*UBP30302064

@dingman
Copy link
Author

dingman commented Oct 19, 2017

@troymccabe @Flet touching base...any updates on this?

@dingman
Copy link
Author

dingman commented Jan 16, 2018

@troymccabe @Flet Hi team, hope the holidays were good. Do we have any updates on this issue? Thanks!

@troymccabe
Copy link
Contributor

Haven't heard anything personally since the last message I sent over a while ago. @Flet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants