Skip to content
This repository has been archived by the owner on Nov 15, 2018. It is now read-only.

Configurator iFrame not working #19

Closed
sareenp2 opened this issue May 17, 2018 · 11 comments
Closed

Configurator iFrame not working #19

sareenp2 opened this issue May 17, 2018 · 11 comments

Comments

@sareenp2
Copy link

I recently started using your this app to manage my home assistant and it is truly awesome! I am however having an issue with the configurator addon. I added it to my home assistant frontend using iframe and it is also using SSL along with a username and password. I'm not sure if you can fix it but if you could that would be awesome! Also if you could get html5 notifications working with electron (if that is even possible) that would even cooler!

@matthinc
Copy link
Owner

Hi,
thank you for the feedback. Could you please explain the issue a bit further?
Does it just not show, do you receive an error etc.

@sareenp2
Copy link
Author

It simply says 'no auth header'. I've attached a picture below
untitled

@rudders
Copy link

rudders commented May 17, 2018

The configurator app has its own userid and password. It also times out after a while and there is no way I’ve found for the Elektron based app to get it to ask for the credentials again, it simply returns the auto error. I have this in a hass.io based install.

@rudders
Copy link

rudders commented May 17, 2018

Image

Happens in iOS app also so it’s not this app I don’t believe - it’s the behaviour of the Configurator app.

@matthinc
Copy link
Owner

matthinc commented May 17, 2018

An easy fix would be this (it's already working for me), but the harder part is to show the dialog. Electron doesn't have a library for that :/

app.on('login', function(event, webContents, request, authInfo, callback) {
event.preventDefault()
callback('username','password')
})

@sareenp2
Copy link
Author

Where/how would I be able to implement this?

@matthinc
Copy link
Owner

screen shot 2018-05-17 at 23 41 45

You don't have to - I will add it to the settings. Will be done tomorrow

@sareenp2
Copy link
Author

Awesome, thanks for the quick update!

@rudders
Copy link

rudders commented May 17, 2018

Sweet! Kudos!

@matthinc
Copy link
Owner

@sareenp2 Could you please test this version (It works for me, but I don't have SSL): https://github.com/matthinc/HomeAssistantElectron/releases/tag/conf_test

You can enter the credentials in the preferences.

@sareenp2
Copy link
Author

Yup, it seems to work fine for me!

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

No branches or pull requests

3 participants