Skip to content

Commit 51b08f6

Browse files
committed
lint: Predicate method names should end with ?
1 parent 1cf2763 commit 51b08f6

File tree

3 files changed

+97
-97
lines changed

3 files changed

+97
-97
lines changed

lib/hooks/plugins/auth/hmac.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ def self.valid?(payload:, headers:, config:)
127127
end
128128

129129
# Validate signature format using shared validation but with HMAC-specific length limit
130-
return false unless validate_signature_format(raw_signature)
130+
return false unless validate_signature_format?(raw_signature)
131131

132132
# Now we can safely normalize headers for the rest of the validation
133133
normalized_headers = normalize_headers(headers)
@@ -192,7 +192,7 @@ def self.valid?(payload:, headers:, config:)
192192
# @param signature [String] Raw signature to validate
193193
# @return [Boolean] true if signature is valid
194194
# @api private
195-
def self.validate_signature_format(signature)
195+
def self.validate_signature_format?(signature)
196196
# Check signature length with HMAC-specific limit
197197
if signature.length > MAX_SIGNATURE_LENGTH
198198
log.warn("Auth::HMAC validation failed: Signature length exceeds maximum limit of #{MAX_SIGNATURE_LENGTH} characters")

0 commit comments

Comments
 (0)