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

Change passwordlike to passterm #1548

Merged
merged 2 commits into from
Nov 6, 2017
Merged

Change passwordlike to passterm #1548

merged 2 commits into from
Nov 6, 2017

Conversation

kanes115
Copy link
Contributor

The name passwordlike smells. As it can't be password because it is either binary (word) OR TUPLE of type scram:scram_tuple() we will call it passTERM.

It plays a role of passWORD but instead of being a WORD it is a WORD or TUPLE. The smallest denominator then is TERM. This way we also let somebody extend it for any type of pass phrase he/she needs.

@kanes115
Copy link
Contributor Author

kanes115 commented Nov 2, 2017

OTP 20 build is considered not passed but there is not a single test that failed. Why? (ping: @fenek )

@fenek
Copy link
Member

fenek commented Nov 2, 2017

@kanes115
Copy link
Contributor Author

kanes115 commented Nov 2, 2017

@fenek , I know which job failed - this is inside the job that I don't find any failed tests :(

@fenek
Copy link
Member

fenek commented Nov 2, 2017

Ouch. :D Well, it's restarted already. Sometimes the job may fail if Cover exits with a segfault (very rare but does happen). In such case you won't see any failed test cases.

@fenek fenek merged commit 4f25073 into master Nov 6, 2017
@fenek fenek deleted the passwordlike_type branch November 6, 2017 13:12
@fenek fenek added this to the 2.1.1 milestone Dec 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants