User may be able to set 'system' style swift location
Bug #1334196 reported by
Stuart McLaren
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Glance |
Fix Released
|
High
|
Stuart McLaren |
Bug Description
This change:
https:/
Introduces a new system style swift scheme: swift+config
A new function "validate_location" verifies that that scheme is not being set by a user
when using the 'set location' functionality.
However, that function will only perform that check if the default backend is swift.
If the swift store is enabled but the default store is 'ceph' say then the base
version of that function (which performs no checking) will be called.
I think 'validate_location' should probably be removed and a check against 'swift+config' should
be performed in _validate_source, in the same way as 'file' is checked there.
Changed in glance: | |
status: | New → Confirmed |
Changed in glance: | |
assignee: | nobody → Jesse J. Cook (jesse-j-cook) |
Changed in glance: | |
assignee: | Jesse J. Cook (jesse-j-cook) → Stuart McLaren (stuart-mclaren) |
tags: | added: juno-rc-potential |
Changed in glance: | |
importance: | Undecided → High |
Changed in glance: | |
milestone: | none → juno-rc2 |
tags: | removed: juno-rc-potential |
Changed in glance: | |
milestone: | juno-rc2 → 2014.2 |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/107383
Review: https:/