forked from loafle/openapi-generator-original
[Ruby] Client: fix base_url when no server_operation_index is defined (#15162)
As discussed in https://github.com/OpenAPITools/openapi-generator/pull/7415#discussion_r1113274416, it seems unlikely the code was correct. server_operation_index is a hash table. In Ruby, `hash[key]` will return the value associated with `key`. If key is absent, `nil` is returned. Because that is sometimes undesirable, there is also `hash.fetch(key)`, which raises an error if the key is absent. It also allows you to specify a default to fall back on: `hash.fetch(key, default)` will return `default` if the key is absent. So, since not all users will specify a 'server per operation' (or at least: I'm not), the old code would usually set `index` to the `server_index`, which is initialized to 0. The subsequent `if index == nil` will usually return false (`0 != nil` in Ruby), after which the `server_url` call on line 177 constructs the url based on the `server_operation_variables` and `operation_server_settings`, assuming we are dealing with the case where a server per operation is configured. The case where the url should be constructed from `scheme`, `host`, etc. is only called if either `server_index` is explicitly set to `nil` or the key `operation` is explicitly associated with the value `nil` in the `server_operation_index` hash table, both of which seem inappropriate.
This commit is contained in:
parent
e6c64d3917
commit
2679819694
@ -171,7 +171,7 @@ module {{moduleName}}
|
||||
|
||||
# Returns base URL for specified operation based on server settings
|
||||
def base_url(operation = nil)
|
||||
index = server_operation_index.fetch(operation, server_index)
|
||||
index = server_operation_index[operation]
|
||||
return "#{scheme}://#{[host, base_path].join('/').gsub(/\/+/, '/')}".sub(/\/+\z/, '') if index == nil
|
||||
|
||||
server_url(index, server_operation_variables.fetch(operation, server_variables), operation_server_settings[operation])
|
||||
|
@ -200,7 +200,7 @@ module Petstore
|
||||
|
||||
# Returns base URL for specified operation based on server settings
|
||||
def base_url(operation = nil)
|
||||
index = server_operation_index.fetch(operation, server_index)
|
||||
index = server_operation_index[operation]
|
||||
return "#{scheme}://#{[host, base_path].join('/').gsub(/\/+/, '/')}".sub(/\/+\z/, '') if index == nil
|
||||
|
||||
server_url(index, server_operation_variables.fetch(operation, server_variables), operation_server_settings[operation])
|
||||
|
@ -204,7 +204,7 @@ module Petstore
|
||||
|
||||
# Returns base URL for specified operation based on server settings
|
||||
def base_url(operation = nil)
|
||||
index = server_operation_index.fetch(operation, server_index)
|
||||
index = server_operation_index[operation]
|
||||
return "#{scheme}://#{[host, base_path].join('/').gsub(/\/+/, '/')}".sub(/\/+\z/, '') if index == nil
|
||||
|
||||
server_url(index, server_operation_variables.fetch(operation, server_variables), operation_server_settings[operation])
|
||||
|
@ -200,7 +200,7 @@ module Petstore
|
||||
|
||||
# Returns base URL for specified operation based on server settings
|
||||
def base_url(operation = nil)
|
||||
index = server_operation_index.fetch(operation, server_index)
|
||||
index = server_operation_index[operation]
|
||||
return "#{scheme}://#{[host, base_path].join('/').gsub(/\/+/, '/')}".sub(/\/+\z/, '') if index == nil
|
||||
|
||||
server_url(index, server_operation_variables.fetch(operation, server_variables), operation_server_settings[operation])
|
||||
|
@ -200,7 +200,7 @@ module XAuthIDAlias
|
||||
|
||||
# Returns base URL for specified operation based on server settings
|
||||
def base_url(operation = nil)
|
||||
index = server_operation_index.fetch(operation, server_index)
|
||||
index = server_operation_index[operation]
|
||||
return "#{scheme}://#{[host, base_path].join('/').gsub(/\/+/, '/')}".sub(/\/+\z/, '') if index == nil
|
||||
|
||||
server_url(index, server_operation_variables.fetch(operation, server_variables), operation_server_settings[operation])
|
||||
|
@ -200,7 +200,7 @@ module DynamicServers
|
||||
|
||||
# Returns base URL for specified operation based on server settings
|
||||
def base_url(operation = nil)
|
||||
index = server_operation_index.fetch(operation, server_index)
|
||||
index = server_operation_index[operation]
|
||||
return "#{scheme}://#{[host, base_path].join('/').gsub(/\/+/, '/')}".sub(/\/+\z/, '') if index == nil
|
||||
|
||||
server_url(index, server_operation_variables.fetch(operation, server_variables), operation_server_settings[operation])
|
||||
|
@ -200,7 +200,7 @@ module Petstore
|
||||
|
||||
# Returns base URL for specified operation based on server settings
|
||||
def base_url(operation = nil)
|
||||
index = server_operation_index.fetch(operation, server_index)
|
||||
index = server_operation_index[operation]
|
||||
return "#{scheme}://#{[host, base_path].join('/').gsub(/\/+/, '/')}".sub(/\/+\z/, '') if index == nil
|
||||
|
||||
server_url(index, server_operation_variables.fetch(operation, server_variables), operation_server_settings[operation])
|
||||
|
Loading…
x
Reference in New Issue
Block a user