openstax_accounts 9.7.0 → 9.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 57482982ec158c16ae7ad70aadbf9bda911cd56b27fa4bfdab86c74c6ce008b2
-  data.tar.gz: fb98bd455ce7f4758d7a862305e83048b8809b1548d1bd87eaa596f433a8f820
+  metadata.gz: 3cd4aa7a22ea69ad3166b90650c73a63b0bcdf5b029bb6d1d3405d45c3769fdc
+  data.tar.gz: 4d0308a4b059dc5d4f81042995d930f60a571c44f6bdf432f9718b771f198b75
 SHA512:
-  metadata.gz: c9a7247488af4a6a96bbb8bb81ead9ea2208efd896a0590a4767c2c7ba88913a8deec5ef9661b72ac15d66f0be489c5177c28c277c527f65e923c904b1bb3038
-  data.tar.gz: aa45a18b1bc00ed0db4f6ab13ca85ea54dcb25be4a00dc356d202e3d00033cc8162b1c039e86f5a0c0b0932102c04e6cb657a830b20c2963b6a722c6adf0aa6c
+  metadata.gz: 9594b3dfa88ac6e1a79ab3c75e13a895eea7661827050c61f7573983a44696a69a29d0433dc23937a3ff18d745ec1e52e15dba01682be7d095ff2dcc5aaa62cd
+  data.tar.gz: d808626f78220265082eaac3855b4f47700c843d0e36564ff67e4e3e6194790fca1c9497aab2093ee1e8a8da8387f41603ee9ee9c2fdfe36caeaa7f247d04516

data/app/controllers/openstax/accounts/sessions_controller.rb

@@ -47,7 +47,7 @@ module OpenStax
 
       def profile
         # TODO: stub profile if stubbing is enabled
-        redirect_to URI.join(configuration.openstax_accounts_url, '/profile').to_s
+        redirect_to URI.join(configuration.openstax_accounts_url, 'profile').to_s
       end
     end
   end

data/lib/omniauth/strategies/openstax.rb

@@ -4,11 +4,11 @@ require 'omniauth-oauth2'
 module OmniAuth
   module Strategies
     class Openstax < OAuth2
-      option :name, "openstax"
+      option :name, 'openstax'
 
       option :client_options, {
-        site: "http://accounts.openstax.org",
-        authorize_url: "/oauth/authorize"
+        authorize_url: 'oauth/authorize',
+        token_url: 'oauth/token'
       }
 
       uid { raw_info[:uuid] }
@@ -21,9 +21,8 @@ module OmniAuth
       extra { { raw_info: raw_info } }
 
       def raw_info
-        @raw_info ||= access_token.get('/api/user.json').parsed.symbolize_keys
+        @raw_info ||= access_token.get('api/user.json').parsed.symbolize_keys
       end
-
     end
   end
 end

data/lib/openstax/accounts/api.rb

@@ -1,7 +1,6 @@
 module OpenStax
   module Accounts
     module Api
-
       DEFAULT_API_VERSION = :v1
 
       # Executes an OpenStax Accounts API request,
@@ -113,10 +112,11 @@ module OpenStax
         @client ||= OAuth2::Client.new(
           OpenStax::Accounts.configuration.openstax_application_id,
           OpenStax::Accounts.configuration.openstax_application_secret,
-          site: OpenStax::Accounts.configuration.openstax_accounts_url
+          site: OpenStax::Accounts.configuration.openstax_accounts_url,
+          authorize_url: 'oauth/authorize',
+          token_url: 'oauth/token'
         )
       end
-
     end
   end
 end

data/lib/openstax/accounts/configuration.rb

@@ -6,8 +6,12 @@ module OpenStax
       attr_reader :openstax_accounts_url
 
       def openstax_accounts_url=(url)
-        url.gsub!(/https|http/,'https') if !(url =~ /localhost/)
-        url = url + "/" if url[url.size-1] != '/'
+        # Only localhost urls are allowed to not be https
+        url.sub!('http:', 'https:') unless url.start_with?('http://localhost')
+
+        # We use URL.join with this url and it needs to end in / to make relative paths work
+        url += '/' unless url.end_with?('/')
+
         @openstax_accounts_url = url
       end
 
@@ -100,7 +104,7 @@ module OpenStax
       end
 
       def default_logout_redirect_url
-        URI.join(openstax_accounts_url, "logout").to_s
+        URI.join(openstax_accounts_url, 'logout').to_s
       end
 
       attr_writer :return_to_url_approver
@@ -114,7 +118,7 @@ module OpenStax
       def initialize
         @openstax_application_id = 'SET ME!'
         @openstax_application_secret = 'SET ME!'
-        @openstax_accounts_url = 'https://accounts.openstax.org/'
+        @openstax_accounts_url = 'https://openstax.org/accounts'
         @enable_stubbing = true
         @logout_via = :get
         @default_errors_partial = 'openstax/accounts/shared/attention'

data/lib/openstax/accounts/version.rb

@@ -1,5 +1,5 @@
 module OpenStax
   module Accounts
-    VERSION = '9.7.0'
+    VERSION = '9.8.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openstax_accounts
 version: !ruby/object:Gem::Version
-  version: 9.7.0
+  version: 9.8.0
 platform: ruby
 authors:
 - JP Slavinsky
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-08 00:00:00.000000000 Z
+date: 2021-11-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -378,7 +378,7 @@ files:
 homepage: http://github.com/openstax/accounts-rails
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -393,8 +393,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.6
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Rails common code and bindings for the 'accounts' API
 test_files: []