Aggregate ID validation

Bug #1855776 reported by Mykola Yakovliev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
In Progress
Undecided
Mykola Yakovliev

Bug Description

Description
===========
Nova API's aggregate ID lookup does not require an exact match. Alphanumeric strings can possibly be truncated and converted to integers incorrectly.

Steps to reproduce
==================
Determine the ID of an existing aggregate.

Take the aggregate ID, and append junk data to it, ensuring that the digit following the actual ID is an alphabetic character.e.g. aggregate id = '13', the test string would be something like '13a2g152asdf'Send a PUT request to '<NOVA API>/os_aggregates/<modified aggregate ID>,' modifying either the name or availability zone

Check for whether or not the server returns an error (aggregate ID not found), or a 200 OK indicating the change was successful.
Successful change indicates the issue still exists.

Expected result
===============
Nova should return error.

Actual result
=============
Nova returns 200.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.opendev.org/698094

Changed in nova:
assignee: nobody → Mykola Yakovliev (vegasq)
status: New → In Progress
Revision history for this message
Doug Schveninger (ds6901) wrote :

This bug also exist in the show command

Revision history for this message
Johannes Kulik (jkulik) wrote :

This should be fixed by merging https://review.opendev.org/#/c/710214/
The bug was not closed, as I was only aware of a duplicate ticket https://bugs.launchpad.net/nova/+bug/1865040

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.