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

Bug #1497319 reported by Kyrylo Romanenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel Plugins
Fix Committed
Medium
Max Yatsenko

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: ldap
Revision history for this message
Kyrylo Romanenko (kromanenko) wrote :
summary: - [ldap] lack of input validation in ldap plugin settings
+ [fuel-plugin-ldap] lack of input validation in ldap plugin settings
Vasyl Saienko (vsaienko)
Changed in fuel-plugins:
assignee: nobody → Vasyl Saienko (vsaienko)
Revision history for this message
Kyrylo Romanenko (kromanenko) wrote :

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

Changed in fuel-plugins:
status: New → Confirmed
Changed in fuel-plugins:
assignee: Vasyl Saienko (vsaienko) → Max Yatsenko (myatsenko)
milestone: none → 7.0
Revision history for this message
Vasyl Saienko (vsaienko) wrote :

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 :
Changed in fuel-plugins:
importance: Undecided → Medium
status: Confirmed → Fix Committed
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.