check MAX_ACCOUNT_NAME_LENGTH in all requests

Bug #1609231 reported by zheng yin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Confirmed
Low
zheng yin

Bug Description

MAX_ACCOUNT_NAME_LENGTH is configurable and therefore could be changed, and made smaller, during the life of a cluster. That means there could be account names that exist in a cluster that are longer than the current constraint but can be accessed with GET/HEAD/POST。
Therefore, it only checks account name length in PUT.

Tags: proxy-server
zheng yin (yin-zheng)
Changed in swift:
assignee: nobody → zheng yin (yin-zheng)
summary: - check MAX_CONTAINER_NAME_LENGTH in all requests
+ check MAX_ACCOUNT_NAME_LENGTH in all requests
Changed in swift:
importance: Undecided → Low
Revision history for this message
Thiago da Silva (thiagodasilva) wrote :

MAX_CONTAINER_NAME_LENGTH is currently checked in GET/HEAD/POST/PUT, it should definitely remain in PUT and POST, but it is not clear why it was added for GET/HEAD, maybe someone with more history knows and can make an argument for keeping/removing.

tags: added: proxy-server
Revision history for this message
Thiago da Silva (thiagodasilva) wrote :
Revision history for this message
John Dickinson (notmyname) wrote :

the fix is to move the constraint check (see the IRC conversation), and this is correctly categorized as low priority

Changed in swift:
status: New → Confirmed
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.