aiji42 / next-fortress

This is a Next.js plugin that redirects or rewrite for accesses that are not authenticated.

https://next-fortress.vercel.app

MIT License

45 stars 4 forks source link

fix: update how cookies are read and extracted from NextCookies #209

Closed shainegordon closed 2 years ago

shainegordon commented 2 years ago

The Cognito auth check is not working correctly as it assume the cookies are an object. They are infact a Map, and therefore you cannot use Object.entries

vercel[bot] commented 2 years ago

Someone is attempting to deploy a commit to a Personal Account owned by @aiji42 on Vercel.

@aiji42 first needs to authorize it.

codecov-commenter commented 2 years ago

Codecov Report

Merging #209 (a24d318) into main (ca2b1f1) will decrease coverage by 0.54%. The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              main     #209      +/-   ##
===========================================
- Coverage   100.00%   99.45%   -0.55%     
===========================================
  Files            6        6              
  Lines          180      183       +3     
  Branches        55       56       +1     
===========================================
+ Hits           180      182       +2     
- Misses           0        1       +1

Impacted Files	Coverage Δ
src/cognito.ts	`98.21% <100.00%> (-1.79%)`	:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

aiji42 commented 2 years ago

@shainegordon, I merge this PR as it passed the test without any problems.
v5.0.1 will be released immediately with the merge.

Your contribution is greatly appreciated.

github-actions[bot] commented 2 years ago

:tada: This PR is included in version 5.0.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: