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

[Konnected] Setting auto discovered callback URL in configuration #13295

Merged
merged 1 commit into from
Aug 21, 2022

Conversation

haavar
Copy link
Contributor

@haavar haavar commented Aug 20, 2022

Setting the auto discovered callback URL in the thing configuration. This will expose the user to the discovered URL, instead of an empty form field.

In my case the auto discovered callback URL is incorrect, and I had to dig into code / debug logs to troubleshoot. If I was exposed to the discovered callback URL in the thing settings, I may have found the issue earlier.

The auto discovered callback URL is incorrect when using reverse proxy or TLS.

Change also includes general cleanup of callback URL handling and method / variable naming.

…lback URL handling

Signed-off-by: Haavar Valeur <haavar@haavar.com>
@jlaur jlaur added the enhancement An enhancement or new feature for an existing add-on label Aug 20, 2022
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks!

@jlaur jlaur merged commit ed3642d into openhab:main Aug 21, 2022
@jlaur jlaur added this to the 3.4 milestone Aug 21, 2022
leifbladt pushed a commit to leifbladt/openhab-addons that referenced this pull request Oct 15, 2022
…lback URL handling (openhab#13295)

Signed-off-by: Haavar Valeur <haavar@haavar.com>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
…lback URL handling (openhab#13295)

Signed-off-by: Haavar Valeur <haavar@haavar.com>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
…lback URL handling (openhab#13295)

Signed-off-by: Haavar Valeur <haavar@haavar.com>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
marcelGoerentz pushed a commit to marcelGoerentz/openhab-addons that referenced this pull request Nov 14, 2022
…lback URL handling (openhab#13295)

Signed-off-by: Haavar Valeur <haavar@haavar.com>
psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Feb 23, 2023
…lback URL handling (openhab#13295)

Signed-off-by: Haavar Valeur <haavar@haavar.com>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Feb 28, 2023
…lback URL handling (openhab#13295)

Signed-off-by: Haavar Valeur <haavar@haavar.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants