Skip to content

Bun Compatibility & API Key Authorisation - #2

Open
r4yfon wants to merge 7 commits into
hnthap:mainfrom
r4yfon:main
Open

Bun Compatibility & API Key Authorisation #2
r4yfon wants to merge 7 commits into
hnthap:mainfrom
r4yfon:main

Conversation

@r4yfon

@r4yfon r4yfon commented Dec 23, 2025

Copy link
Copy Markdown
  • added bun compatibility so that commands for bun can be run instead of npm
  • added authorisation for api to allow only certain calls to work. mitigates usage concerns

@hnthap

hnthap commented Dec 28, 2025

Copy link
Copy Markdown
Owner

@r4yfon

Hi! Thanks for the contribution. I am interested in the API Key authentication, but I cannot merge this as is.

Could you please make the following changes?

  1. Revert the Bun changes: Please revert package.json and .gitignore to use npm and Node.js. I do not want to switch the project to Bun.
  2. Make Auth Optional: Can you update apiKeyAuth.ts so that if no API_KEY environment variable is set, the API remains open? It should only enforce auth if the variable exists.
  3. Remove the file Sending: It looks like an accidental commit.

Let me know if you can make these updates!

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