collective / collective/Products.AutoUserMakerPASPlugin

manage_delProperties in ExtractionPlugin.getConfig

Open
#13 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
3
Forks
6
PR merge metrics
No merged PRs in 30d

Description

manage_delProperties expects a collection of IDs but only a string is provided. Therefore an exception complaining about a non-existent property with a name of length 1 is raised when an entry is removed from the configuration:

> 2017-05-05 12:41:40 ERROR Zope.SiteErrorLog 1493980900.90.779300729568 ...
> Traceback (innermost last):
> Module ZPublisher.Publish, line 127, in publish
> Module ZPublisher.BaseRequest, line 607, in traverse
> Module Products.PluggableAuthService.PluggableAuthService, line 244, in validate
> Module Products.PluggableAuthService.PluggableAuthService, line 605, in _extractUserIds
> Module Products.AutoUserMakerPASPlugin.auth, line 542, in extractCredentials
> Module Products.AutoUserMakerPASPlugin.auth, line 400, in getConfig
> Module OFS.PropertyManager, line 377, in manage_delProperties
> BadRequest: The property f does not exist

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.