8000 normalize url paths before making a request · Issue #1 · go-jira/jira · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

normalize url paths before making a request #1

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
Closed
jaybuff opened this issue Feb 15, 2015 · 3 comments
Closed

normalize url paths before making a request #1

jaybuff opened this issue Feb 15, 2015 · 3 comments

Comments

@jaybuff
Copy link
Contributor
jaybuff commented Feb 15, 2015

https://issues.apache.org/jira//rest/api/2/issue/MESOS-1123 returns a 404, but https://issues.apache.org/jira/rest/api/2/issue/MESOS-1123 works :(

I had this problem with this config:

$ cat ~/.jira.d/config.yml
endpoint: https://issues.apache.org/jira/

Perhaps you can simply strip any trailing slashes from endpoint if they exists.
@jaybuff
Copy link
Contributor Author
jaybuff commented Feb 15, 2015

I looked, net/url doesn't have a normalize function (bummer). I found https://github.com/PuerkitoBio/purell but it's just a bunch of regexes and it's complete overkill.

@coryb
Copy link
Contributor
coryb commented Feb 16, 2015

It might be easier to just trim off any trailing / when I read the endpoint property from the config files, I try to find something.
Thanks,
-Cory

coryb added a commit that referenced this issue Feb 16, 2015
@coryb
[issue #1] trim trailing / on uri
3a0867d
@coryb
Copy link
Contributor
coryb commented Feb 16, 2015

I just went with a simple TrimRight, seems to work.
-Cory

@coryb coryb closed this as completed Feb 16, 2015
pdericson pushed a commit to pdericson/go-jira that referenced this issue Sep 14, 2017
@coryb
[issue go-jira#1] trim trailing / on uri
798016d
coryb pushed a commit that referenced this issue Nov 30, 2018
@ejsuncy
Fix grammar typos (#1)
9258d4d
coryb added a commit that referenced this issue Nov 30, 2018
@coryb
Merge pull request #220 from ejsuncy/master
514c149 
docs: Fix grammar typos (#1)
@hkyeakley hkyeakley mentioned this issue Oct 24, 2019
Open
5D3B
jenic pushed a commit to jenic/go-jira that referenced this issue Feb 6, 2025
@prpht9
Merge pull request go-jira#1 from prpht9/jira_9_createmeta_api_url_bug
7c4bee0 
fix createmeta issue on jira 9.x
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jaybuff @coryb
0