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.

Sign up for GitHub

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

Jump to bottom

Alpha9 #150

Merged

sserrata merged 4 commits into PaloAltoNetworks:master from sserrata:alpha9

Apr 3, 2020

Member

sserrata commented Apr 3, 2020

Description

Check if static access_token exists before developer token refresh, fixes Refresh incorrectly attempted when both static access_token and developer_token are present #147
Reduce number of times json() property is called, fixes Cleanup unnecessary use of json() property #148
Update clientType and clientVersion, fixes Incorrect clientType and clientVersion defined in create_query() method #149

Motivation and Context

Addresses bugs reported by partners and other general cleanup.

How Has This Been Tested?

Tested locally with scripts
CI/CD
pytest

Screenshots (if appropriate)

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

I have updated the documentation accordingly.
I have read the CONTRIBUTING document.
I have added tests to cover my changes if appropriate.
All new and existing tests passed.

sserrata added 4 commits

April 2, 2020 11:03


          Update clientType and clientVersion, fixes PaloAltoNetworks#149

89c94d5


          Reduce number of times json() property is called, fixes PaloAltoNetwo…

8243ec5

…rks#148


          Check if static access_token exists before developer token refresh, f…

00b7921

…ixes PaloAltoNetworks#147


          Bump version to alpha9

a1abd19

sserrata merged commit d3709b3 into PaloAltoNetworks:master

sserrata deleted the alpha9 branch

April 3, 2020 17:51

sserrata mentioned this pull request

Subsequent refresh() attempts fail when using developer_token #153

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet