Skip to content

Ruby 2.2+ compatibility#10

Open
methodmissing wants to merge 7 commits into
onelogin:masterfrom
Shopify:master
Open

Ruby 2.2+ compatibility#10
methodmissing wants to merge 7 commits into
onelogin:masterfrom
Shopify:master

Conversation

@methodmissing
Copy link
Copy Markdown

References #8

See Shopify#1 for further context.

@jtdowney
Copy link
Copy Markdown

jtdowney commented Jun 13, 2016

Wouldn't this break compatibility with ruby < 2.2? Since in those versions the cipher context is just data and not typed data.

@methodmissing
Copy link
Copy Markdown
Author

It'll break - I'll PR a backwards compatible change. Thx for flagging.

@methodmissing
Copy link
Copy Markdown
Author

@jtdowney inbound via Shopify#2 - feel free to drop comments there.

@methodmissing
Copy link
Copy Markdown
Author

methodmissing commented Jun 14, 2016

Merged the PR upstream. Review bump with the full changeset that also is backwards compat as per comments from @jtdowney in Shopify#2 (comment)

@jtdowney
Copy link
Copy Markdown

I am not sure who has the ability to merge this and bump the version but @Lordnibbler owns the rubygem.

@Lordnibbler
Copy link
Copy Markdown
Contributor

@jtdowney I do not have the ability to merge any longer, you would need someone at OneLogin to do this for you. I am the owner of the gem on Rubygems and could cut a new release, but ideally someone at OneLogin should take ownership and handle this in the future.

Let me try and ping some folks:
@tinifni @prateekm21 @chunkychode @christianbpedersen

@ryanong
Copy link
Copy Markdown

ryanong commented Sep 27, 2017

@Lordnibbler I think at this point it might be time to fork the source and give it to someone IE shopify to maintain.

LONG LIVE THE MIT LICENSE

@Lordnibbler
Copy link
Copy Markdown
Contributor

@ryanong sorry I wish I could but I haven’t worked with onelogin since 2015, so I’m no longer an owner :(

@ryanong
Copy link
Copy Markdown

ryanong commented Sep 27, 2017

I mean as the push owner of the gem, you could change the source of the gem.

@tibbon
Copy link
Copy Markdown

tibbon commented Apr 25, 2018

And status update here? Or is this dead in the water?

@Lordnibbler
Copy link
Copy Markdown
Contributor

I no longer have write access to this Gem, and although Rubygems thinks I am the owner/author I no longer have access to any machines with credentials to change owners.

Someone from Onelogin would have to help. I'm not entirely sure how to tell who can do this.

@jtdowney
Copy link
Copy Markdown

I am shooting in the dark here, but I saw that @pitbulk pushed to onelogin/ruby-saml recently so perhaps they know how to get this either a) maintained at onelogin or b) transferred so the community can support it.

@pitbulk
Copy link
Copy Markdown

pitbulk commented Apr 25, 2018

Hi, I will try to ask at onelogin who can do it. (I pushed ruby-saml with my user, not with onelogin account that is the owner)

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.

7 participants