-
Notifications
You must be signed in to change notification settings - Fork 60
Spool QR code points to http instead of https #156
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
Comments
hmm..is there a config-setting in OctoPrint I missed? |
I found an topic in the OP-Forum: https://community.octoprint.org/t/reverse-proxy-configuration-examples/1107
|
fyi: I ask the community: |
Hi @jkaessens, |
Actually it's a super plain haproxy stanza:
I feel that forwarding the existing parameters via HTTP headers (like in the HAProxy forum topic) is not how it's supposed to work. After all, the end point should already know everything from existing headers. I might be wrong though, and try the suggested workaround to see if it also works for SpoolManager. I'll check back with you on that. |
Hi @jkaessens, I used a newer version of haproxy, so the the statement looks like this (instead of
|
I seem to have the opposite issue, I get |
- PR #204, #201 redesign of the Spool-Selection in the sidebar. Huge thanks to @TiziG @MarloSM - E #125 New field "Serialnumber" - E Spool selection from Edit-Dialog - E #141 New Filter: Material, Vendor and Color - E #44 Sidebar filament usage in grams instead of volume - B #199, #198, #163, #156 some QR-Code optimisations - B #197, #196, #188, #181, #178, #177, #172, #162, #142, Some UI optimisations - B #194 CSV import, behind reverse proxy - B #175 Confirm dialog not shown
Hi @normen, @jkaessens please also test in your environment it it still works Thx in advance |
Yes, that seems to work now, thanks! |
Yes, thank you! Works perfectly. |
The spool QR code contains
http
even if the UI is loaded viahttps
. Octoprint is run through HAproxy, so flask can't possibly know about the real URL. It's probably safer pick the correct URL from the actual HTTP request header, or have a config option to override Octoprint's base URL.The text was updated successfully, but these errors were encountered: