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

Help Wanted #1393

Open
1 of 9 tasks
bmulholland opened this issue Dec 26, 2021 · 20 comments
Open
1 of 9 tasks

Help Wanted #1393

bmulholland opened this issue Dec 26, 2021 · 20 comments

Comments

@bmulholland
Copy link
Contributor

bmulholland commented Dec 26, 2021

As most have noticed, and previously flagged, this project needs more attention. I have just put in a few days over Christmas to upgrade all the packages and ensure the demo app is working, but still improvements are needed. This is all table stakes for a maintained app, and yet I have limited time to do it myself. I am asking for help with any of these tasks, which would be the most immediately helpful in moving the project toward a more stable and bug-free (hah!) state.

If you make any improvement on any of these, even partially, please create a PR and assign me for review.

Happy to answer questions, or even make time for a short call to discuss anything.

Tasks:

  • Get chore(deps): update devdependency oidc-provider to v7 #1306 green
  • In dev-mode demo app (yarn dev), fix Login with Google
  • Write instructions for setting up the dev-mode demo app (yarn dev) to test Facebook
  • In dev-mode demo app (yarn dev), fix Login with Github
  • Write instructions for setting up the dev-mode demo app (yarn dev) to test Laravel (Sanctum, JWT, Passport, and Password Grant -- all at once, or one at a time)
  • Improve test coverage: even just one test at a time would be appreciated
  • Make it simpler to add tests: write documentation on how they work, refactor so there's less boilerplate
  • Test, report, and fix issues with nuxt bridge and nuxt 3: not work with nuxt 3 #1323
  • Review TODOs/FIXMEs in the app and either: create a PR with a fix or removing it (with justification), or create a Github Issue with next steps
@Intevel
Copy link
Contributor

Intevel commented Jan 13, 2022

Github Auth works for me on the demo page. @bmulholland

@Intevel
Copy link
Contributor

Intevel commented Jan 13, 2022

Login with Google also works for me on the demo page.

@bmulholland
Copy link
Contributor Author

bmulholland commented Jan 14, 2022

@Intevel Thanks for the help! What about when run locally with yarn dev?

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

@Intevel Thanks for the help! What about when run locally with yarn dev?

Oh no, I have now tested only Github but it does not work or I get a 404 error on Github? Do you mean that? @bmulholland

@bmulholland
Copy link
Contributor Author

Yep, that's what I mean. We need to be able to test changes locally for new development/changes/refactors.

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

Yes that would be good, I just wonder why we get a 404 error from Github.

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

Okay, it does not add the client_id to the paramenters. That's why we get a 404 error

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

But Google works in the Development Mode of the demo. @bmulholland

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

I reloaded the project and wrote my oAuth2 data directly into the config without .env, Github works too.

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

Sorry for commenting so much right now, I'm trying to understand. So I don't get 404 error anymore, as long as I write the data directly to nuxt.config.ts everything works, as soon as I use .env the callback doesn't work anymore and it loads all the time n

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

Now I don't understand anything anymore, Google doesn't work now either. 😨

@Intevel
Copy link
Contributor

Intevel commented Jan 14, 2022

okay, that's stupid. Github now it works again after I restarted the demo.

@bmulholland
Copy link
Contributor Author

Could you please create a specific ticket to track this work and document the results of your experiments into the README as a PR? I can help clean up if needed. That will help keep this specific conversation separate from the broader Help Wanted topic.

@andrewdleach
Copy link

@bmulholland, does this work help us get auth working with Nuxt3? (i'm historically BE oriented, but am more than happy to help out if more hands are needed)

@bmulholland
Copy link
Contributor Author

No, the nuxt 3 version is a complete rewrite

@andrewdleach
Copy link

Do you happen to know where that work is being tracked @bmulholland? I'm new-er to Nuxt development and would love any pointers to docs that you find helpful

@bmulholland
Copy link
Contributor Author

It's being worked on privately first

@th1nkgr33n
Copy link

Hey @bmulholland believing rumors the nuxt3 migration happens in a private beta :D ?
My Team and I need to make some decisions and maybe you are willing to give us some hints.
At the point we have an nuxt2 project (using this auth module) as well as a new nuxt3 project starting now.

The nuxt3 project needs oauth as well and we have looked forward that this module would be part of the stable release.
Things often don't turn out the way you planned, but can you tell me when this module would be migrated?
Otherwise we need to choose another one (maybe just as interim solution) and come back later to this, but I only want to do this, if you are telling me for example that your hitting the finish linie not earlier than in a couple of weeks/months.

On the other hand we have a team internal discussion around to support the migration process. Is this possible?

Thanks for response and have a nice day!

@bmulholland
Copy link
Contributor Author

bmulholland commented Dec 1, 2022

I'm not involved in the rewrite and, as far as I am aware, there's no clear timeline. My company is also -- in part -- blocked from upgrading to Nuxt 3 because of this module. Our plans to upgrade include migrating off of nuxt/auth.

@th1nkgr33n
Copy link

@bmulholland thanks for response! Good luck for migration!

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

No branches or pull requests

4 participants