mirror of
https://github.com/PowerDNS-Admin/PowerDNS-Admin.git
synced 2026-04-25 00:35:52 +03:00
[GH-ISSUE #105] AD authentication has no email address for gravatar #68
Labels
No labels
bug / broken-feature
bug / broken-feature
docs / request
feature / request
help / other
mod / help-wanted
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/PowerDNS-Admin-PowerDNS-Admin#68
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 @ihamburglar on GitHub (Aug 15, 2016).
Original GitHub issue: https://github.com/PowerDNS-Admin/PowerDNS-Admin/issues/105
I installed and hooked up to AD.
Once you log in using your AD account
It looks like the culprit is the code that turns an email into a gravatar via the email address in utils.py. However when authing with AD you don't tell the app about your email address.
@joachimtingvold commented on GitHub (Aug 15, 2016):
Should probably extend the config to have an
LDAP_EMAILFIELDoption, in addition to checking if the email-value is None (set it to '' rather than None or something).@ihamburglar commented on GitHub (Aug 15, 2016):
On the other hand... The my profile section allows you to enter your own email address, which might be better for those who want to use the email address associated with their gravatar and not their corporate email address. Unless you wanted to default to whatever was in AD and then let the user overwrite.
@joachimtingvold commented on GitHub (Aug 15, 2016):
Either works.
I have my corporate email added to the same Gravatar-account as my personal, so, I'm kinda "whatever" :-P