Skip to content

Authentication Bypass by CSRF Weakness

Critical severity GitHub Reviewed Published Nov 17, 2021 in spree/spree_auth_devise • Updated May 26, 2023

Package

bundler spree_auth_devise (RubyGems)

Affected versions

= 4.2.0

Patched versions

4.2.1

Description

Impact

CSRF vulnerability that allows user account takeover.

All applications using any version of the frontend component of spree_auth_devise are affected if protect_from_forgery method is both:

  • Executed whether as:
    • A before_action callback (the default)
    • A prepend_before_action (option prepend: true given) before the :load_object hook in Spree::UserController (most likely order to find).
  • Configured to use :null_session or :reset_session strategies (:null_session is the default in case the no strategy is given, but rails --new generated skeleton use :exception).

That means that applications that haven't been configured differently from what it's generated with Rails aren't affected.

Thanks @waiting-for-dev for reporting and providing a patch 👏

Patches

Spree 4.3 users should update to spree_auth_devise 4.4.1
Spree 4.2 users should update to spree_auth_devise 4.2.1

Workarounds

If possible, change your strategy to :exception:

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception
end

Add the following toconfig/application.rb to at least run the :exception strategy on the affected controller:

config.after_initialize do
  Spree::UsersController.protect_from_forgery with: :exception
end

References

GHSA-xm34-v85h-9pg2

References

@damianlegawiec damianlegawiec published to spree/spree_auth_devise Nov 17, 2021
Reviewed Nov 17, 2021
Published to the GitHub Advisory Database Nov 18, 2021
Last updated May 26, 2023

Severity

Critical
9.3
/ 10

CVSS base metrics

Attack vector
Network
Attack complexity
Low
Privileges required
None
User interaction
Required
Scope
Changed
Confidentiality
High
Integrity
High
Availability
None
CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:H/I:H/A:N

Weaknesses

CVE ID

No known CVE

GHSA ID

GHSA-6mqr-q86q-6gwr

Credits

Checking history
See something to contribute? Suggest improvements for this vulnerability.