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

Version 1.5.3 breaks 1.9.3 compatibility, but not documented as such #132

Closed
BanzaiMan opened this issue Feb 25, 2016 · 6 comments
Closed
Assignees
Labels
Milestone

Comments

@BanzaiMan
Copy link

Yes, I know 1.9.3 has been EOL'd many months ago. It is OK if this is a carefully considered decision. But https://github.com/jwt/ruby-jwt/blob/v1.5.3/ruby-jwt.gemspec does not indicate it.

It should be either indicated as such, or reinstate 1.9.x support.

Upon require, it fails:

$ pry
[1] pry(main)> require 'jwt'
SyntaxError: /Users/asari/.rvm/gems/ruby-1.9.3-p551/gems/jwt-1.5.3/lib/jwt/verify.rb:7: unknown type of %string
      %i[verify_aud verify_expiration ...
         ^
/Users/asari/.rvm/gems/ruby-1.9.3-p551/gems/jwt-1.5.3/lib/jwt/verify.rb:7: syntax error, unexpected $end
      %i[verify_aud verify_expiration ...
         ^
from /Users/asari/.rvm/rubies/ruby-1.9.3-p551/lib/ruby/site_ruby/1.9.1/rubygems/core_ext/kernel_require.rb:54:in `require'
@danleyden
Copy link
Contributor

I wonder if the bump to 1.5.3 from 1.5.2 should have been a major bump
(breaking backwards-compatibility) per semantic versioning?

On 25 February 2016 at 03:52, Hiro Asari notifications@github.com wrote:

Yes, I know 1.9.3 has been EOL'd many months ago. It is OK if this is a
carefully considered decision. But
https://github.com/jwt/ruby-jwt/blob/v1.5.3/ruby-jwt.gemspec does not
indicate it.

It should be either indicated as such, or reinstate 1.9.x support.

Upon require, it fails:

$ pry[1] pry(main)> require 'jwt'SyntaxError: /Users/asari/.rvm/gems/ruby-1.9.3-p551/gems/jwt-1.5.3/lib/jwt/verify.rb:7: unknown type of %string %i[verify_aud verify_expiration ... ^/Users/asari/.rvm/gems/ruby-1.9.3-p551/gems/jwt-1.5.3/lib/jwt/verify.rb:7: syntax error, unexpected $end %i[verify_aud verify_expiration ... ^from /Users/asari/.rvm/rubies/ruby-1.9.3-p551/lib/ruby/site_ruby/1.9.1/rubygems/core_ext/kernel_require.rb:54:in `require'


Reply to this email directly or view it on GitHub
#132.

@anthony-battaglia
Copy link

Encountered same issue. Currently forcing 1.5.2 version in Gemfile. Version bump should have denoted a breaking change.

@excpt
Copy link
Member

excpt commented Mar 23, 2016

I made 2 PRs to fix the 1.5.3 issues. Mentioned in #133.

PRs: #135 #136

If you find the time - you can check out the master branch and provide some feedback.

@excpt excpt added the bug label Mar 24, 2016
@excpt excpt added this to the Version 1.5.4 milestone Mar 24, 2016
@excpt excpt self-assigned this Mar 24, 2016
@excpt
Copy link
Member

excpt commented Mar 24, 2016

Version 1.5.3 yanked.

@excpt
Copy link
Member

excpt commented Mar 24, 2016

Version 1.5.4 released.

@excpt excpt closed this as completed Mar 24, 2016
@sentience
Copy link

👋 Just dropping by to mention that the yanking of 1.5.3 broke our builds, since the version in our Gemfile.lock is no longer available. I’d suggest that this be considered more carefully before yanking versions post-release in future.

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