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

fix(middleware): Context declaration #2694

Closed
wants to merge 2 commits into from

Conversation

MonsterDeveloper
Copy link
Contributor

Fixes #2693

@yusukebe not sure it's the best way though, as denoify automatically adds node: to hono import... Without this import TS yells that hono is an "unknown" module...

The author should do the following, if applicable

  • Add tests (n/a)
  • Run tests
  • bun denoify to generate files for Deno
  • bun run format:fix && bun run lint:fix to format the code

@yusukebe
Copy link
Member

Hi @MonsterDeveloper thank you for the PR!

About the PR, can you wait for me to merge? Perhaps it will be fixed completely if Hono supports JSR.

src/middleware/jwt/index.ts Outdated Show resolved Hide resolved
src/middleware/secure-headers/index.ts Outdated Show resolved Hide resolved
src/middleware/secure-headers/index.ts Outdated Show resolved Hide resolved
src/middleware/timing/index.ts Outdated Show resolved Hide resolved
@yusukebe
Copy link
Member

@MonsterDeveloper

I got it. We should extend ../.. instead of ../../context. Could you fix them?

@MonsterDeveloper
Copy link
Contributor Author

Oh my lovely TypeScript:)

Corrected to ../... Seems to be working too.

@MonsterDeveloper
Copy link
Contributor Author

But now Deno is not happy...
CleanShot 2024-05-18 at 06 02 53@2x

@yusukebe
Copy link
Member

@MonsterDeveloper

Oh... Then, wait until this PR is merged. The next minor version may be released soon, and it will be merged at that time!

@MonsterDeveloper
Copy link
Contributor Author

@yusukebe hi! Seems like #2662 is merged now, can we resume the work on this PR?

@yusukebe
Copy link
Member

The issue was fixed. This PR is not needed. Thanks!

@yusukebe yusukebe closed this May 28, 2024
@MonsterDeveloper MonsterDeveloper deleted the middleware branch May 28, 2024 12:48
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.

Type bug introduced in v4.3.0 in secureHeaders middleware
2 participants