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.

Sign up for GitHub

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

Jump to bottom

Path param encoding ignores forward slashes #315

Closed
Jazzatola opened this issue Mar 25, 2015 · 5 comments
Closed

Path param encoding ignores forward slashes #315

Jazzatola opened this issue Mar 25, 2015 · 5 comments
Assignees
@whitlockjc

Comments

@Jazzatola
Copy link

I'm working on an application using Digital Object Identifiers in path parameters. These identifiers contain slashes (/). Unfortunately the current implementation of encodePathParam in swagger-client.js ignores the slashes when encoding the path parameters. The comment indicates there is some uncertainty about this implementation. Any plans to change it?

Thanks.
@Jazzatola Jazzatola changed the title Path param encoding ignores forward slashes (/) Path param encoding ignores forward slashes Mar 25, 2015
@fehguy
Copy link
Contributor

fehguy commented Mar 26, 2015

@whitlockjc can you comment on this?

@whitlockjc
Copy link
Contributor

Can you tell me if this is still happening with the latest release?

@fehguy
Copy link
Contributor

fehguy commented May 7, 2015

see #280

Need feedback from @webron

@webron
Copy link
Contributor

webron commented May 9, 2015

Gave the feedback in #280.

@webron
Copy link
Contributor

webron commented Jun 24, 2015

Based on #280, I'll assume this is done. Please reopen if needed.

@webron webron closed this as completed Jun 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@whitlockjc whitlockjc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Jazzatola @whitlockjc @webron @fehguy