feat: rest-express-auth example

[gautvm]

This is the PR where I am working on the rest-express-auth example, details #2838. This is currently a draft.

[ruheni]

Hi @eternalmoon1234 👋🏽

Do you have any updates on this PR? Would you like any help pushing this across the finish line? 🙂

[ruheni]

Hey @eternalmoon1234 👋🏽

Do you intend to pick up this PR, and do you need any help?

[gautvm]

Hey @ruheni, ✋
Yes, I do intend to pick up the PR and will finish it by the end of next week! Sorry for the inactivity with this, I've been coding on and off recently and I must say I forgot about this 😅

Looking forward to finishing this!

[gautvm]

Hey @ruheni 👋

I finally finished up this PR. I've implemented the changes you proposed (as well as some others) and have followed the same standard of the rest-express-auth example.

Please review & propose any changes when you can!

[ruheni]

This looks pretty good. I've left some comments. The example seems to be missing a test for it. You can add one in the .github/tests folder.

One other thing I didn't include in the comments is that I think some endpoints might not require authentication, such as /feed and /post/:id (both GET endpoints). I would recommend protecting endpoints with write operations in general. Let me know if you would like any help determining what endpoints to protect

[gautvm]

Hey @ruheni 👋

I've incorporated all of your requested changes. As for the protected routes, I've modified the ones that I thought required protection based on your guidance, although please feel free to suggest any alterations if you'd propose something different.

Please let me know if there's anything else you'd like me to change 🙂

[gautvm]

Do I need to update the rest test and postman collection now that there are protected routes?

[ruheni]

Hey 👋🏾
I would recommend creating a new test and collection for it to get the tests to pass.

[gautvm]

Sounds good, I'll go ahead and write the new test in.

[gautvm]

Hey @ruheni 👋

The tests should pass now. Mind running the checks & verifying that everything's all good? If so, I think this is ready to merge.

[XprabhatX]

The logs for the failed test are no longer available. Can anyone specify what's the problem.