Skip to content

Commit fa6af0b

Browse files
committed
fix rubocop linting issues
1 parent e2164c8 commit fa6af0b

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

.rubocop.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,11 @@ Metrics/CyclomaticComplexity:
2929
Metrics/PerceivedComplexity:
3030
Max: 17
3131

32+
Naming/PredicateMethod:
33+
AllowedMethods:
34+
- "validate_origin"
35+
- "remove_resource"
36+
3237
# Style configuration
3338
Style/Documentation:
3439
Enabled: false

lib/mcp/transports/rack_transport.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ def call(env)
136136

137137
private
138138

139-
def validate_client_ip(request)
139+
def valid_client_ip?(request)
140140
client_ip = request.ip
141141

142142
# Check if we're in localhost-only mode
@@ -204,7 +204,7 @@ def extract_hostname(url)
204204
# Handle MCP-specific requests
205205
def handle_mcp_request(request, env)
206206
# Validate client IP to ensure it's connecting from allowed sources
207-
return forbidden_response('Forbidden: Remote IP not allowed') unless validate_client_ip(request)
207+
return forbidden_response('Forbidden: Remote IP not allowed') unless valid_client_ip?(request)
208208

209209
# Validate Origin header to prevent DNS rebinding attacks
210210
return forbidden_response('Forbidden: Origin validation failed') unless validate_origin(request, env)

0 commit comments

Comments
 (0)