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

Docker support and API Key enabled. #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lojik-ng
Copy link

Docker support makes it easily deployable without nodejs installation and API Key makes it more secure when used over tunnels.

	modified:   README.md
	new file:   docker-compose.yml
	modified:   package-lock.json
	deleted:    pnpm-lock.yaml
	modified:   src/controllers/message.ts
	modified:   src/controllers/session.ts
	modified:   src/env.ts
	modified:   src/index.ts
	modified:   src/middlewares/key.middleware.ts
@mimamch
Copy link
Owner

mimamch commented Dec 13, 2024

Love this, but actually we already have API key feature, and it is as long as optional.

And do not write env on env.ts. You would use regular .env instead

@lojik-ng
Copy link
Author

Oh. Thanks for the update. I thought the api key wasnt implemented.
Kudos for the great job you did!

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