audit fixes: two latent NameErrors in the org code — webhook org branch referenced undefined org_name (crashed on an unlinked org's first payment), org-links GET referenced undefined name (crashed when a linked org's dues show inactive)
This commit is contained in:
1 parent
d87cbf9650
commit
fed4a7005e
1 file changed
+2
-2
+2
-2
@@ -2009,7 +2009,7 @@ async def opencollective_webhook(request: Request, token: str):
|
|||||||
slug, org_email)
|
slug, org_email)
|
||||||
if first_payment:
|
if first_payment:
|
||||||
try:
|
try:
|
||||||
await provision_member(org_email, org_name, slug)
|
await provision_member(org_email, name, slug)
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.warning("Webhook: org re-provision failed for %s: %s",
|
logger.warning("Webhook: org re-provision failed for %s: %s",
|
||||||
org_email, e)
|
org_email, e)
|
||||||
@@ -2417,7 +2417,7 @@ async def admin_org_links(request: Request):
|
|||||||
org = org_backers.get(org_slug)
|
org = org_backers.get(org_slug)
|
||||||
links.append({
|
links.append({
|
||||||
"org_slug": org_slug,
|
"org_slug": org_slug,
|
||||||
"org_name": (org or {}).get("name") or name,
|
"org_name": (org or {}).get("name") or org_slug,
|
||||||
"email": email,
|
"email": email,
|
||||||
"account_active": bool(active),
|
"account_active": bool(active),
|
||||||
"dues_active": org is not None,
|
"dues_active": org is not None,
|
||||||
|
|||||||
Reference in new issue
Block a user