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

Update api-consumer.jsx #18

Merged
merged 1 commit into from Aug 23, 2018
Merged

Conversation

foxns1
Copy link
Contributor

@foxns1 foxns1 commented Aug 23, 2018

While the webUI is accessible via the network hardcoding 'localhost' returns errors when uploading from another machine. Substituting "localhost' with 'window.location.hostname' dynamically changes the POST URL path to adjust and remain functional from other machines on the Network. In my case Xori can now be used on a centralized server that is shared between team members.

Dynamically adapt the PORT URL to support network access of file uploading.
@CLAassistant
Copy link

CLAassistant commented Aug 23, 2018

CLA assistant check
All committers have signed the CLA.

@rseymour
Copy link
Contributor

We were just talking about a good way to fix this. Thanks. I hope you can sign our CLA!

@foxns1
Copy link
Contributor Author

foxns1 commented Aug 23, 2018

Glad to help, sorry for the CLA!

@rseymour
Copy link
Contributor

just tested it out and it works well. thanks!

I may soon add a warning about running the xori gui on networks being "at your own risk". I imagine most users would be well aware, but you never know who might try it and be surprised at strangers disassembling things on their machine, or worse trying to exploit the js servers.

@rseymour rseymour merged commit ceed7f7 into endgameinc:master Aug 23, 2018
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.

None yet

3 participants