HTTP Error 500: Internal Server Error when calling get_audit_log with "after" variable

Open

#101 opened on Nov 7, 2021 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 3/5
Estimated time: 1-2 days
Newbie friendliness: 42/100

Issue type: Bug
Clarity: Mostly clear
Activity status: Stale
Tech stack: github, python
Domain: api

Research direction

Start in /opt/conda/lib/python3.9/site-packages/ghapi/core.py and compare how get_audit_log builds the request with and without the after argument. Check the GitHub audit-log API documentation and reproduce the supplied call using the documented date and timestamp forms; done means the supported format is handled correctly or the invalid value produces a clear error, with the expected format documented.

Written by the indexing model from the issue text.

Description

api.orgs.get_audit_log(ORG, per_page=10, page=2, after="2021-09-27T15:05:06+01:00")
// Here I'm using time format as per GHAPI rest docs. ghapi python docs suggest 2021-09-27 should work. Tried
// this and also timestamps, but all gave internal error.

If i remove the "after" variable, the call works fine.

I believe the problem is that the "after" variable is in [Link header format](api.orgs.get_audit_log(ORG, per_page=10, page=2, after="2021-09-27T15:05:06+01:00")) and perhaps this doesn't get autogenerated correctly.

Link: https://api.github.com/orgs/ORG/audit-log?after=MTYwMTkxOTU5NjQxM3xZbGI4VE5EZ1dvZTlla09uWjhoZFpR&before=; rel="next",

The value between after= and & does not completely URL + base64 decode to an obvious time format.

Signature:  
api.orgs.get_audit_log(
    org,
    phrase=None,
    include=None,
    after=None,
    before=None,
    order=None,
    per_page=None,
    page=None,
)
Type:        _GhVerb
String form:
orgs.get_audit_log(org, phrase=None, include=None, after=None, before=None, order=None, per_page=None, page=None)
https://docs.github.com/rest/reference/orgs#get-audit-log
File:        /opt/conda/lib/python3.9/site-packages/ghapi/core.py

I would expect any values in the signature to specify the time format required, possibly add an example to the docs, and the error should be something less severe than a 500 internal error.

Dominant language: Python
Stars: 687
Forks: 69
Avg merge: 1m
Merged PRs (30d): 2

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

[Bug] reef-hermes tells me to resume with hermes --resume, which does not work from my shell Open

area: harness bug status: needs-triage

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

Human-Agent-Society/reef#625 · Opened Sep 24, 2026
Bump transitive axios to >=1.18.0 to address GHSA-hcpx-6fm6-wx23 Open

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

coinbase/cdp-sdk#828 · Opened Sep 24, 2026
AccessibleResourceCard/AccessibleChannelCard crash when a resource has more than one learning activity (getActivityTags returns an object instead of an array) Open

Difficulty 1/5 Under an hour Newbie friendliness 80/100

learningequality/kolibri#15351 · 2 comments · Opened Sep 24, 2026
audeer.path(): optionally expand environment variables Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

audeering/audeer#212 · 1 comment · Opened Sep 24, 2026
Name consistency Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

eellak/triplestore#65 · 1 comment · Opened Sep 24, 2026

More Python issues