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

New server example #243

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

New server example #243

wants to merge 4 commits into from

Conversation

beothorn
Copy link

This PR would adds a new example to the examples folder.
This is a simple express js server that saves alexa cookies under home/.alexaRemote

I think this would be a good addition, as it shows what needs to be persisted to recover the session and also shows a setup that you can just start using right away.

@Apollon77
Copy link
Owner

@beothorn I basically like the idea, thank you, my "issue" is that it would add "express" as a dependency just for the example which is not needed for the normal operation, so I would see this more as a separate thing - or have the example but without having express and having info that users need to install it additionally ... or we put it just as a devDependency?

WDYT?

@beothorn
Copy link
Author

The example is supposed to be a separate thing indeed. This is why I created it as if it was a new project inside the examples folder.
I think it is confusing because I did not add any explanation, I will add a README with instructions on how to build it.
The dependencies and the whole setup is exclusive to the example.

As for the reason I did it with express, I was a bit confuse about the cookies format and how could I persist it, both between executions and also inside the running process.
Since there are not that many options to have something that keeps running and you can interact with I went with this setup. It is also what I currently need for my private home server so I thought two birds with one stone.

If you still think it is not a good idea to have this separate setup with expressjs I can modify the example to still save and recover from files and remove the expressjs, it would still show how to persist the session, just not show you can keep it alive in a long runnin process.

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

Successfully merging this pull request may close these issues.

2 participants