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

Added Covertix SmartCipher default URLs #123

Merged
merged 1 commit into from May 12, 2014
Merged

Added Covertix SmartCipher default URLs #123

merged 1 commit into from May 12, 2014

Conversation

ghost
Copy link

@ghost ghost commented May 12, 2014

Added Covertix SmartCipher default URLs based on this documentation:

http://www.kagoon.com/smartcipher-installation-guide-docx/main

This documentation also contains the default credentials for the test 006745:

Username: admin
Password: Admin

but i'm unsure how to add this. In a few weeks i can get my hands on a live system so i could provide better return patterns than "200".

@ghost
Copy link
Author

ghost commented May 12, 2014

Also not sure if the /manage and /ws URLs should be changed to something more generic / to @CGIDIRS

sullo added a commit that referenced this pull request May 12, 2014
Added Covertix SmartCipher default URLs
@sullo sullo merged commit 4ef1039 into sullo:master May 12, 2014
@sullo
Copy link
Owner

sullo commented May 12, 2014

The only way to add auth in a db_tests entry is via the headers (assuming it's Basic or a static cookie). I'll add the pass to the db on cirt.net though.

I wouldn't worry too much about having variables for /manage and /ws unless we see a lot of entries in both (which is where the benefit really kicks in).

@ghost
Copy link
Author

ghost commented May 12, 2014

I wouldn't worry too much about having variables for /manage and /ws unless we see a lot of entries in both (which is where the benefit really kicks in).

Ok, thanks for the info.

I'll add the pass to the db on cirt.net though.

Thanks, just seen it. I'm not absolutely sure if the case matters in this case but its: admin/Admin instead of Admin/Admin.

@ghost ghost deleted the cxs_tests branch May 13, 2014 06:46
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.

None yet

1 participant