andrei-tatar / node-red-contrib-nora

Node Red Google Home integration
74 stars 25 forks source link

Enable a not code based url for nora-service #41

Open mabels opened 4 years ago

mabels commented 4 years ago

added the functionality to get the service url from the token

Xumpy commented 3 years ago

Due to the recent problems with bandwith I have installed my own server on heroku. I have noticed 2 problems.

My solution was to remove that oauth.ts validation. I have a commit for this in my forked repo. I think that people would be very interested in hosting their own nora-server.

Changes on the server side are not that imprtant for me but due to my own changes in Nora.ts under the nodered module I'm not able to update the connector anymore. Can you tell me if or when this hardcoded url will be removed?Thanks for the hard work Andrei!

andrei-tatar commented 3 years ago

@Xumpy if you mean on this line: https://github.com/andrei-tatar/nora-service/blob/875f7a81b5f3eb804c4ebf7f58b348863cb05b26/src/http/controllers/oauth.ts#L55 It's based on the firebase project id. It should always be in that format.

I'm waiting on google's approval on the new firebase based service. I don't plan on doing any further development work on the previous version.