Commit 9cf652fe authored by Rajendra Kadam's avatar Rajendra Kadam

Unmask scope and state in url masking

parent e3b07c65
...@@ -3,7 +3,10 @@ ...@@ -3,7 +3,10 @@
module Routing module Routing
module PseudonymizationHelper module PseudonymizationHelper
class MaskHelper class MaskHelper
QUERY_PARAMS_TO_NOT_MASK = %w[].freeze QUERY_PARAMS_TO_NOT_MASK = %w[
scope
state
].freeze
def initialize(request_object, group, project) def initialize(request_object, group, project)
@request = request_object @request = request_object
......
...@@ -160,7 +160,7 @@ RSpec.describe ::Routing::PseudonymizationHelper do ...@@ -160,7 +160,7 @@ RSpec.describe ::Routing::PseudonymizationHelper do
end end
context 'when author_username is present' do context 'when author_username is present' do
let(:masked_url) { "http://localhost/dashboard/issues?author_username=masked_author_username&scope=masked_scope&state=masked_state" } let(:masked_url) { "http://localhost/dashboard/issues?author_username=masked_author_username&scope=all&state=opened" }
let(:request) do let(:request) do
double(:Request, double(:Request,
path_parameters: { path_parameters: {
...@@ -201,7 +201,7 @@ RSpec.describe ::Routing::PseudonymizationHelper do ...@@ -201,7 +201,7 @@ RSpec.describe ::Routing::PseudonymizationHelper do
end end
context 'when query string has keys with the same names as path params' do context 'when query string has keys with the same names as path params' do
let(:masked_url) { "http://localhost/dashboard/issues?action=masked_action&scope=masked_scope&state=masked_state" } let(:masked_url) { "http://localhost/dashboard/issues?action=masked_action&scope=all&state=opened" }
let(:request) do let(:request) do
double(:Request, double(:Request,
path_parameters: { path_parameters: {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment