Fuel Plugins

[fuel-plugin-ldap] lack of input validation in ldap plugin settings

Bug #1497319 reported by Kyrylo Romanenko on 2015-09-18

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Fuel Plugins	Fix Committed	Medium	Max Yatsenko	Fuel Plugins 7.0

Bug Description

1. Deployed Fuel environment.
2. Installed LDAP plugin.
3. Created a cluster by wizard.
4. Enable "LDAP plugin for Keystone" in Settings.
5. Fill invalid values to fields:
LDAP domain
LDAP URL
6. Save Settings.
Expected to have validation to prevent from invalid settings.

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "7.0"
  openstack_version: "2015.1.0-7.0"
  api: "1.0"
  build_number: "298"
  build_id: "298"
  nailgun_sha: "d590b26dbb09785b8a8b3651b0ef69746fcf9991"
  python-fuelclient_sha: "486bde57cda1badb68f915f66c61b544108606f3"
  fuel-agent_sha: "082a47bf014002e515001be05f99040437281a2d"
  fuel-nailgun-agent_sha: "d7027952870a35db8dc52f185bb1158cdd3d1ebd"
  astute_sha: "6c5b73f93e24cc781c809db9159927655ced5012"
  fuel-library_sha: "0623b4daad438ceeb5dc41b10cdd3011795fff7e"
  fuel-ostf_sha: "1f08e6e71021179b9881a824d9c999957fcc7045"
  fuelmain_sha: "6b83d6a6a75bf7bca3177fcf63b2eebbf1ad0a85"

fuel-plugin-ldap built on commit 80845952bb0bb9e9ddfb43ad007b1fd9222c5fe9.

Tags:

Revision history for this message

Kyrylo Romanenko (kromanenko) wrote on 2015-09-18:

Selection_088.png Edit (191.3 KiB, image/png)

Kyrylo Romanenko (kromanenko) on 2015-09-21

summary:

- [ldap] lack of input validation in ldap plugin settings
+ [fuel-plugin-ldap] lack of input validation in ldap plugin settings

Vasyl Saienko (vsaienko) on 2015-09-21

Changed in fuel-plugins:
assignee:	nobody → Vasyl Saienko (vsaienko)

Revision history for this message

Kyrylo Romanenko (kromanenko) wrote on 2015-10-01:

Fuel Dashboard - LDAP - Google Chrome_105.png Edit (176.6 KiB, image/png)

Bug is still reproduced with updated plugin from 30th September build.

Oleksiy Butenko (obutenko) on 2015-10-01

Changed in fuel-plugins:
status:	New → Confirmed

Irina Povolotskaya (ipovolotskaya) on 2015-11-04

Changed in fuel-plugins:
assignee:	Vasyl Saienko (vsaienko) → Max Yatsenko (myatsenko)
milestone:	none → 7.0

Revision history for this message

Vasyl Saienko (vsaienko) wrote on 2015-11-04:

Validation for the following fields is added https://review.openstack.org/#/c/230360/

* LDAP domain
* LDAP URL
* LDAP User Password

Revision history for this message

Max Yatsenko (myatsenko) wrote on 2015-12-11:

https://review.openstack.org/#/c/230360/ was merged.

Changed in fuel-plugins:
importance:	Undecided → Medium
status:	Confirmed → Fix Committed

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Bug attachments

Add attachment

Remote bug watches

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