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

external javascript and stylesheet loading not https friendly #194

Open
danp opened this issue Jan 3, 2017 · 5 comments
Open

external javascript and stylesheet loading not https friendly #194

danp opened this issue Jan 3, 2017 · 5 comments
Labels

Comments

@danp
Copy link

danp commented Jan 3, 2017

When trying to serve the OBA webapp via https things break because external javascript and stylesheet resources are hardcoded to http://... instead of the more friendly //....

For example the Google Maps resources here.

@barbeau barbeau added the bug label Jan 3, 2017
@barbeau
Copy link
Member

barbeau commented Jan 3, 2017

Thanks @danp! Pull requests welcome if you've already fixed this ;).

@danp
Copy link
Author

danp commented Jan 3, 2017

I have a start to fix things in onebusaway-application-modules which I'll push up. Think the lone holdout is this from gwt-search. Not sure how to fix that.

@barbeau
Copy link
Member

barbeau commented Jan 3, 2017

Cool, thanks! Eek - yeah, GWT might pose problems. It's never been one to cooperate and AFAIK the plugin has been abandoned by Google.

You might want to give this PR a spin for a newer web UI:
#176

...under onebusaway-enterprise-acta-webapp. This will eventually (hopefully soon) replace the legacy web UI, which is the current onebusaway-webapp. Here's the new web UI in action in Tampa:
http://tampa.onebusaway.org/enterprise/#Hillsborough Area Regional Transit_7800

@danp
Copy link
Author

danp commented Jan 5, 2017

Will check out the new UI PR, thanks!

Opened #195 to fix direct javascript loading in webapp at least.

@barbeau
Copy link
Member

barbeau commented Jan 5, 2017 via email

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

Successfully merging a pull request may close this issue.

2 participants