search

Tufin / pytos

A Python SDK for Tufin Orchestration Suite
https://www.tufin.com
Apache License 2.0
55 stars 21 forks source link

Fix get_app_by_name() returning wrong app #10

Closed geewrd closed 6 years ago

geewrd commented 6 years ago

Added app.name equality condition for code path where app_domain is provided, and a new list comprehension with app.name equality for the other code paths.

An IndexError is still raised if the app is not found in all cases.

chrisdotcode commented 6 years ago

@geewrd LGTM. Thanks for this.

@funkycode, This has been tested that it is working, let's merge this when we get a chance.