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

webui, url localhost:5001/webui wrong #291

Closed
flowpoint opened this issue Oct 12, 2017 · 3 comments
Closed

webui, url localhost:5001/webui wrong #291

flowpoint opened this issue Oct 12, 2017 · 3 comments
Assignees
@lidel
Labels
exp/novice Someone with a little familiarity can pick up good first issue Good issue for new contributors kind/bug A bug in existing code (including security flaws) P2 Medium: Good to have, but can wait until someone steps up P3 Low: Not priority right now

Comments

@flowpoint
Copy link

when accessing webui via, "localhost:5001/webui", i land at the url but with port 8080.
so: http://127.0.0.1:8080/ipfs/hash/#/
instead of http://127.0.0.1:5001/ipfs/hash/#/
using go-ipfs 0.4.11 as systemservice with the ipfs companion extension 2.0.12
@flowpoint flowpoint mentioned this issue Oct 12, 2017
Closed
@lidel lidel self-assigned this Oct 12, 2017
@lidel lidel added kind/bug A bug in existing code (including security flaws) exp/novice Someone with a little familiarity can pick up good first issue Good issue for new contributors P3 Low: Not priority right now labels Oct 12, 2017
@lidel lidel mentioned this issue Dec 4, 2017
Merged
@lidel
Copy link
Member

lidel commented Dec 4, 2017 •
edited
Loading

This is because extension thinks 127.0.0.1 and localhost are different things.

What if we normalize 127.0.0.1 on saving preferences and always replace it with localhost ?
It improves the Location Bar a bit; looks less scary for non-technical users.

@lidel lidel added the P2 Medium: Good to have, but can wait until someone steps up label Dec 4, 2017
@lidel lidel closed this as completed in 27c6d76 Dec 4, 2017
@lidel
Copy link
Member

lidel commented Dec 4, 2017

@flowpoint No one picked this up, so I've fixed it in v2.1.1beta3, give it a try :)

@lidel lidel mentioned this issue Dec 9, 2017
Closed
@flowpoint
Copy link
Author

flowpoint commented Dec 16, 2017 •
edited
Loading

fixed in v2.1.1beta1 for (tested)
chrome 63.0.3239.84 (Official Build) (64-bit)
firefox 57.0.1 (64-bit)
👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lidel lidel

Labels
exp/novice Someone with a little familiarity can pick up good first issue Good issue for new contributors kind/bug A bug in existing code (including security flaws) P2 Medium: Good to have, but can wait until someone steps up P3 Low: Not priority right now
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lidel @flowpoint