lessonly / scim_rails

SCIM Adapter for Rails.
MIT License
68 stars 76 forks source link

Rails 7? #52

Open dhanson358 opened 2 years ago

dhanson358 commented 2 years ago

The Problem

I notice the gemspec here is limiting to Rails < 6.2

I'm working on a Rails 7 project, and I was wondering if there was anything known/specific that would cause incompatibility with Rails 6.2+, or if it's just not been widely tested?

stevegrossi commented 2 years ago

I'm not aware of any known incompatibilities. We're not on Rails 7 yet so it just hasn't been tested with that version of the framework. Let's leave this issue open to track Rails 7 compatibility.

set-pfel commented 2 years ago

I am having the same issue on my end. Bundler found conflicting requirements for the Ruby version: In Gemfile: scim_rails (> 0.4.0) was resolved to 0.4.0, which depends on Ruby (> 2.4)

Current Ruby version: Ruby (= 3.1.2)

I'm trying to build an Azure AD Integration.

I was able to workaround it by: gem fetch scim_rails gem install --force scim_rails-0.4.0.gem

thedayisntgray commented 2 years ago

Hey @set-pfel,

Similarly my team is needing to use this project in a Rails 7 app. We decided to fork the project for now and maintain it in house at the moment. @tknzk did some work on this too. If you look in the open prs they've done some work to help with compatibility. I also left a comment on their pr about the ruby version currently being used.

@stevegrossi once I get this integrated into our rails 7 application, what would you need from me to get that pr merged in? We would prefer to continue to work off of this project if possible.

set-pfel commented 2 years ago

Hey @thedayisntgray I have abandoned the try to get this gem working with Microsoft Azure. Instead I started from scratch with the REST client gem to synchronize Users, Groups, Managers and Profile pictures from Microsoft Azure via Microsoft Graph.