API Rate limiting should return 429 not 413

Bug #1187676 reported by Phil Day
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
High
Jiajun Liu

Bug Description

Currently the rate limiting code returns 413 (request too large) - it would be better if it returned 429 (too many requests)

Recording so it can be fixed as part of V3 API

Tags: api
tags: added: api
Revision history for this message
Christopher Yeoh (cyeoh-0) wrote :

Looks like the current code is returning 403 not 413, but agree it should be 429

Changed in nova:
status: New → Confirmed
importance: Undecided → High
Jiajun Liu (ljjjustin)
Changed in nova:
assignee: nobody → Jiajun Liu (ljjjustin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/37383

Changed in nova:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/37383
Committed: http://github.com/openstack/nova/commit/a8b8ebb32a184c752495d11fdf47ab12c2af97f3
Submitter: Jenkins
Branch: master

commit a8b8ebb32a184c752495d11fdf47ab12c2af97f3
Author: Jiajun Liu <email address hidden>
Date: Tue Jul 16 10:18:35 2013 +0000

    return 429 on API rate limiting occur

    Currently nova will return 413 instead of 429 on API rate limiting.
    Fix it so that the V3 API will not be influenced.

    fixes bug 1187676

    Change-Id: I25e3f6766597c2475b4fc3203a43b159f0e99739

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.