-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Relax certificate test - was dependent on build-machine timezone #45
Conversation
In my London-based JVM, the date was actually Apr 19th, and the asserted string did not match.
Probably your result is showing there's something timezone-related wrong here, so we might better try and fix it to always give the GMT date in the error message. |
Yup - rather than relaxing the test, the message can become always GMT (and with a well-specified, non-locale dependent formatting), and then test won't be TimeZone/Locale specific - I can submit another PR with that if you like? |
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
In my London-based JVM, the date was actually Apr 19th, and the asserted string in NistCertPathTest did not match. This is an alternative to this prior pull-request, which just relaxed the test condition to ignore the date: bcgit#45 (comment)
Dropped in favour of #48 , thanks again! |
In my London-based JVM, the date was actually Apr 19th, and the asserted string did not match.