Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix searching by unique identifier #205

Merged
merged 4 commits into from
Jun 26, 2020
Merged

Fix searching by unique identifier #205

merged 4 commits into from
Jun 26, 2020

Conversation

razdob15
Copy link
Contributor

  • Search only the unique user/group in nipyapi.security functions.
  • Functions' documentation updated accordingly.

@coveralls
Copy link

coveralls commented Jun 24, 2020

Coverage Status

Coverage decreased (-0.6%) to 66.323% when pulling 824a545 on razdob15:fix-searching-by-unique-identifier into e8a53b7 on Chaffelson:master.

@Chaffelson Chaffelson added this to the 0.15.0 milestone Jun 26, 2020
@Chaffelson Chaffelson linked an issue Jun 26, 2020 that may be closed by this pull request
@Chaffelson
Copy link
Owner

Thanks for this contribution! Rebased onto master and all tests passed, will merge and put into next release.

@Chaffelson Chaffelson merged commit 735d0e2 into Chaffelson:master Jun 26, 2020
rsaggino added a commit to rsaggino/nipyapi that referenced this pull request Mar 29, 2021
* Patch for authentication issue (Chaffelson#193)

bumpversion to 0.14.3
Patch authentication issue to provide control for which authentication method is used in the config. Default is set to tokenAuth per previous behavior.

* Fix create empty group (Chaffelson#207)

* Fixed: Can create a group without pass users list

* Fixed: Can create a group without pass users list #1

* Allow deploy_template to use floats for positions (Chaffelson#191)

* Fix searching by unique identifier (Chaffelson#205)

* Fix the search according unique identifier

* Fix the search according unique identifier and update functions doc

* Minor typo correction

Co-authored-by: Dan Chaffelson <chaffelson@gmail.com>

* Added Parameter Contexts API to docs (Chaffelson#196)

* Added Parameter Contexts API to docs

* omit docs from coverage stats.

* keep track of the docker container object, and use it to kill the container when we are done (Chaffelson#203)

set auto remove to true

* fixed funnel position (Chaffelson#224)

* fixed funnel position, added test and contribution markdown

* removed duplicate file

* add functions for working reading template.xml files into TemplateEnt… (Chaffelson#223)

* add functions for working reading template.xml files into TemplateEntity instances

* undo formatting

* fix import

* fix test assertion and type

* Merge 0.15.0 release from Next to Master (Chaffelson#228)

* Add greedy control to get_processor to support exact matching on get operation
Add basicAuth support to nifi client and security module

* Add greedy control to several get_ functions to support exact matching on operation
Add basicAuth support to nifi client and security module
Updated test containers to 1.11.4
Added special case for 'root' to get_processor
Corrected list_invalid_processors to correctly report an empty list
update_controller now has a refresh option
get_template now correctly returns None if Template not found
is_endpoint_up now has a reasonable default timeout
automatically trim API endpoint URLs of trailing slash to avoid really really annoying errors

* Added configuration control for NiFi Authentication mode, either tokenAuth or basicAuth
Improved Linting and Logging statements in security.py

* Adding vscode to gitignore because OSX is bloody stupid and invasive

* Bump version: 0.14.1 → 0.14.2

* Bump version: 0.14.2 → 0.14.3

* Add greedy control to get_processor to support exact matching on get operation
Add basicAuth support to nifi client and security module

* Added configuration control for NiFi Authentication mode, either tokenAuth or basicAuth
Improved Linting and Logging statements in security.py

* Update base clients to NiFi-1.12.1 and Registry-0.7.0

* Add greedy control to get_processor to support exact matching on get operation
Add basicAuth support to nifi client and security module

* Added configuration control for NiFi Authentication mode, either tokenAuth or basicAuth
Improved Linting and Logging statements in security.py

* Update base clients to NiFi-1.12.1 and Registry-0.7.0

* Updated contributing.rst to include guidance on new features going into Next branch
Added create and delete port to _all_ in canvas.py. Fixes Chaffelson#192
Coding style fixes.

* Bump version: 0.14.3 → 0.15.0

* Doc fixes for release
Update history for release

* Update ruamel.yaml from 0.16.10 to 0.16.12 (Chaffelson#219)

* check-version should ignore snapshot (Chaffelson#232)

* add public api for deserializing models (Chaffelson#236)

* update dump so that it supports dumping swagger entities
this pr also fixes a bug where the nifi style of xml was resulting
in incomplete objects

* update to sync with next branch

Co-authored-by: Dan Chaffelson <chaffelson@gmail.com>
Co-authored-by: Raz <razdob@gmail.com>
Co-authored-by: Bernhard Geisberger <41961259+bgeisberger@users.noreply.github.com>
Co-authored-by: Eric Secules <esecules@gmail.com>
Co-authored-by: Otto Fowler <ottobackwards@gmail.com>
Co-authored-by: Clément Vial <clement.vial@gmail.com>
Co-authored-by: pyup.io bot <github-bot@pyup.io>
rsaggino pushed a commit to rsaggino/nipyapi that referenced this pull request Mar 29, 2021
* Fix the search according unique identifier

* Fix the search according unique identifier and update functions doc

* Minor typo correction

Co-authored-by: Dan Chaffelson <chaffelson@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Logic Error - Identifiers are not unique
3 participants