mirror of
https://github.com/hoppscotch/hoppscotch.git
synced 2026-04-26 01:06:00 +03:00
[GH-ISSUE #3050] [bug]: Self-host unable to use Azure oauth #996
Labels
No labels
CodeDay
a11y
browser limited
bug
bug fix
cli
core
critical
design
desktop
discussion
docker
documentation
duplicate
enterprise
feature
feature
fosshack
future
good first issue
hacktoberfest
help wanted
i18n
invalid
major
minor
need information
need testing
not applicable to hoppscotch
not reproducible
pull-request
question
refactor
resolved
sandbox
self-host
spam
stale
testmu
wip
wont fix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/hoppscotch#996
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @andreufontb on GitHub (May 11, 2023).
Original GitHub issue: https://github.com/hoppscotch/hoppscotch/issues/3050
Is there an existing issue for this?
Current behavior
With a new setup and everything setup to use Microsoft Auth when I try to login the backend server gets an error, it seems "profile" data is empty
After some debugging i've found the problem and editing the file:
/packages/hoppscotch-backend/src/auth/strategies/microsoft.strategy.tsand adding a dummie param to this funcion like this:async validate(accessToken: string, refreshToken: string, dummie, profile, done) {Solves the null problem and the function can access to profile data.
But then it seems prisma gets in to a loop creating the user to the database with this error.
I'm not expert enought to solve this problem by my self and send a PR, sorry...
Steps to reproduce
Environment
Production
Version
Self-hosted
@sydevinfra commented on GitHub (May 15, 2023):
Hello,
Same issue here.
I added the delegated authorizations "email" and "User.ReadBasic.All" just in case (+Admin consent) but I still have the same error message.
Probably unrelated but there is a warning on the App : "Starting November 9th, 2020 end users will no longer be able to grant consent to newly registered multitenant apps without verified publishers."
It should not be an issue since I granted admin consent.
Also probably not a factor, but the site is behind a nginx reverse proxy for https.
@balub commented on GitHub (May 22, 2023):
@andreufontb Can I see the env file you are using?
@andreufontb commented on GitHub (May 23, 2023):
Sure,
I have changed my real company's name for "mycompany" and all secrets and passwords to "XXXXXXXXX"
Hoppscotch is deployed on k8s, and with ingress-nginx as a ingress controller for the custom frontend's internal domain name.
@Saintel commented on GitHub (Jun 13, 2023):
Any news there?
We have the same issue :(
@liyasthomas commented on GitHub (Jun 27, 2023):
Resolved in #3138, hence closing.