Skip to content
This repository was archived by the owner on Dec 16, 2018. It is now read-only.

Conversation

@zecoj
Copy link

@zecoj zecoj commented Mar 6, 2017

vip.symantec.com changed the token check url. This patch fixes it.

vip.symantec.com changed the token check url. This patch fixes it.
@coveralls
Copy link

coveralls commented Mar 6, 2017

Coverage Status

Coverage remained the same at 98.058% when pulling 29f1737 on zecoj:patch-1 into 7f29bef on cyrozap:master.

@dlenski
Copy link

dlenski commented Mar 20, 2017

I confirmed that this works and merged it into dlenski/python-vipaccess as well. Thanks @zecoj!

@ksylvan
Copy link

ksylvan commented May 30, 2017

Does this still work? About to generate some tokens for use in Authy (instead of using the Symantec VIP Access app).

@dlenski
Copy link

dlenski commented May 30, 2017

Does this still work?

My fork (dlenski/python-vipaccess) is definitely still working, and this PR was already merged into it.

@ksylvan
Copy link

ksylvan commented May 30, 2017

@dlenski Thank you. What is the diff between your cli and master branches in your fork? cli seems to be 8 commits ahead and master is 19 commits ahead of the original upstream.

@dlenski
Copy link

dlenski commented May 30, 2017

@ksylvan cli is just an old name I forgot to remove… deleted now.

@ksylvan
Copy link

ksylvan commented May 30, 2017

@dlenski Great. Thanks. That will reduce confusion. I'm going to create a docker container version of this tool (auto-build generated on Docker Hub) so that one can just docker run ... the tool.

@dlenski
Copy link

dlenski commented May 31, 2017

Sounds good. Glad it's useful!

tgage pushed a commit to tgage/python-vipaccess that referenced this pull request Sep 13, 2018
With added commit '2470a0c', which replaces the token type aliases
with a link to Symantec's reference.
@cyrozap
Copy link
Owner

cyrozap commented Dec 16, 2018

Closing this PR since I'm no longer maintaining this project and will be archiving it soon.

@cyrozap cyrozap closed this Dec 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants