Skip to content
This repository has been archived by the owner on Dec 12, 2023. It is now read-only.

Refactor import and use of argon2 #31

Merged
merged 4 commits into from
Nov 15, 2022

Conversation

Voltra
Copy link
Contributor

@Voltra Voltra commented Nov 14, 2022

Now follows the proper usage described in https://github.com/ranisalt/node-argon2#typescript-usage

Closes #30 .

valiafetisov
valiafetisov previously approved these changes Nov 14, 2022
@JacobArrow
Copy link

Just tested this and it seems to work, but I'm still getting an error:
ERROR Vue app aliases are not allowed in server routes. [importing #app from node_modules/@sidebase/nuxt-session/dist/runtime/server/middleware/session/ipPinning.mjs]

Now using the same as the index file
@Voltra
Copy link
Contributor Author

Voltra commented Nov 14, 2022

Just tested this and it seems to work, but I'm still getting an error: ERROR Vue app aliases are not allowed in server routes. [importing #app from node_modules/@sidebase/nuxt-session/dist/runtime/server/middleware/session/ipPinning.mjs]

I'm not well versed in the nuxt+vite magic, but this should do the trick

@BracketJohn
Copy link
Contributor

Thanks @Voltra for the quick help here! @JacobArrow, were you already able to confirm that it works now? Would love to push out this fix soon 💯

@JacobArrow
Copy link

I just tested again and everything seems to work as expected
Thank you @Voltra !

@BracketJohn BracketJohn merged commit 8295cfd into sidebase:main Nov 15, 2022
@BracketJohn
Copy link
Contributor

Fix is merged, release is done, check out:

@JacobArrow please do write here / in the issue if the bug persists ❤️

Thanks again for everybody involved, you're awesome 💯

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

Successfully merging this pull request may close these issues.

Error in ipPinning.mjs middleware
4 participants