Uploaded image for project: 'Archive of Our Own'
  1. Archive of Our Own
  2. AO3-4817

Strong parameters: Admin blacklisted email

    Details

    • Type: Improvement
    • Status: Deployed
    • Priority: Medium
    • Resolution: DeployedToBeta
    • Affects Version/s: 0.9.174
    • Fix Version/s: 0.9.180
    • Required Access Level:
      Admin
    • Milestone:
      Internal 0.9
    • Difficulty:
      Medium
    • Roadmap:
      Admin
    • Components:
      BackEnd

      Description

      Brakeman says:
      Brakeman thinks there is no mass assignment, but we think there might be in the create action:

      @admin_blacklisted_email = AdminBlacklistedEmail.new(params[:admin_blacklisted_email])

      How to test:
      90% automated test coverage. To test manually:

      1. Log in as admin
      2. Follow "Blacklist" link in admin navigation
      3. Enter an email address in "Email to add" field
      4. Press "Add To Blacklist"

        Attachments

          Activity

            People

            • Assignee:
              donat Don Denoncourt (Inactive)
              Reporter:
              sarken sarken
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: