Opened 8 years ago

Closed 8 years ago

#5408 closed defect (fixed)

/api/user/foo/feed?offset=WORSE backtraces

Reported by: Loic Dachary Owned by:
Priority: major Milestone:
Component: programming Keywords:
Cc: Parent Tickets:


  File "/home/mediagoblin/software/mediagoblin/mediagoblin/mediagoblin/api/", line 593, in feed_endpoint
    for activity in outbox:
  File "build/bdist.linux-x86_64/egg/sqlalchemy/orm/", line 2362, in __iter__
    context = self._compile_context()
  File "build/bdist.linux-x86_64/egg/sqlalchemy/orm/", line 2768, in _compile_context
    context.statement = self._simple_statement(context)
  File "build/bdist.linux-x86_64/egg/sqlalchemy/orm/", line 2860, in _simple_statement
  File "build/bdist.linux-x86_64/egg/sqlalchemy/sql/", line 408, in select
  File "build/bdist.linux-x86_64/egg/sqlalchemy/sql/", line 5941, in __init__
    SelectBase.__init__(self, **kwargs)
  File "build/bdist.linux-x86_64/egg/sqlalchemy/sql/", line 5399, in __init__
    self._offset = util.asint(offset)
  File "build/bdist.linux-x86_64/egg/sqlalchemy/util/", line 783, in asint
    return int(value)
ValueError: invalid literal for int() with base 10: 'WORSE'

Attachments (1)

0001-Fix-5408-ignore-non-int-offset-in-api-feed.patch (3.7 KB ) - added by Loic Dachary 8 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 by Loic Dachary, 8 years ago

Owner: set to Loic Dachary
Status: newin_progress

comment:2 by Loic Dachary, 8 years ago

Owner: Loic Dachary removed
Status: in_progressreview

comment:3 by Christopher Allan Webber, 8 years ago

Resolution: fixed
Status: reviewclosed

This looks nice. Committed and pushed. Thank you, loic!

Note: See TracTickets for help on using tickets.