Feature to fetch authentication info from context when auth is enabled

gofr-dev / gofr

An opinionated GoLang framework for accelerated microservice development. Built in support for databases and observability.

https://gofr.dev

Apache License 2.0

3.61k stars 243 forks source link

Feature to fetch authentication info from context when auth is enabled #1159

Closed coolwednesday closed 1 month ago

coolwednesday commented 1 month ago

Pull Request Template

Description:

Added a method GetAuthInfo() in Request Interface to fetch Auth Info when authentication is enabled.
Closes #1094

Checklist:

[x] I have formatted my code using goimport and golangci-lint.
[x] All new code is covered by unit tests.
[x] This PR does not decrease the overall code coverage.
[x] I have reviewed the code comments and documentation for clarity.

Thank you for your contribution!

reski-rukmantiyo commented 2 weeks ago

Hi @coolwednesday, do you have any complete examples for this? Thanks

ccoVeille commented 1 week ago

@coolwednesday why was the PR closed and branch deleted?

coolwednesday commented 1 week ago

Hi @coolwednesday, do you have any complete examples for this? Thanks

Hey @reski-rukmantiyo , you can see the release notes , of release v1.25.0. In case of any doubts, feel free to ping us.

However, the issue mentioned here has been resolved in PR #1161 , for your reference.

coolwednesday commented 1 week ago

@coolwednesday why was the PR closed and branch deleted?

I was facing issue in this branch used in this PR, hence created a new branch and resolved this issue in PR #1161.

ccoVeille commented 1 week ago

Make sense thanks.

It's always a good thing to drop a message when closing a PR.

Also to crossref PR/issues

reski-rukmantiyo commented 1 week ago

Hi @coolwednesday, do you have any complete examples for this? Thanks

Hey @reski-rukmantiyo , you can see the release notes , of release v1.25.0. In case of any doubts, feel free to ping us.

However, the issue mentioned here has been resolved in PR #1161 , for your reference.

Thanks @coolwednesday