Skip to content

Allow 'base_url' (fixes #7784)#7796

Merged
balloob merged 1 commit into
home-assistant:devfrom
fabaff:req-baseurl-upnp
May 31, 2017
Merged

Allow 'base_url' (fixes #7784)#7796
balloob merged 1 commit into
home-assistant:devfrom
fabaff:req-baseurl-upnp

Conversation

@fabaff
Copy link
Copy Markdown
Member

@fabaff fabaff commented May 27, 2017

Description:

Allow the combination of base_url and upnp:.

Related issue (if applicable): fixes #7784

Example entry for configuration.yaml (if applicable):

http:
  base_url: example.duckdns.org:8123
upnp:

Checklist:

If the code communicates with devices, web services, or third-party tools:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass

@mention-bot
Copy link
Copy Markdown

@fabaff, thanks for your PR! By analyzing the history of the files in this pull request, we identified @robbiet480 to be a potential reviewer.

@balloob balloob merged commit 66d6f51 into home-assistant:dev May 31, 2017
@fabaff fabaff deleted the req-baseurl-upnp branch May 31, 2017 20:25
@balloob balloob mentioned this pull request Jun 2, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Sep 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

components/upnp.py stacktrace when http.baseurl populated

5 participants