projects
/
arvados.git
/ blobdiff
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
closes #6277
[arvados.git]
/
services
/
api
/
app
/
models
/
blob.rb
diff --git
a/services/api/app/models/blob.rb
b/services/api/app/models/blob.rb
index 7ae13ef2d0126d0b41b28a344e391ad55a7e6a41..34600d7a25a8c716bd9d1fd6ec49cea052dc0c58 100644
(file)
--- a/
services/api/app/models/blob.rb
+++ b/
services/api/app/models/blob.rb
@@
-28,8
+28,8
@@
class Blob
# Blob.sign_locator: return a signed and timestamped blob locator.
#
# The 'opts' argument should include:
# Blob.sign_locator: return a signed and timestamped blob locator.
#
# The 'opts' argument should include:
- # [required] :
key - the Arvados server-side blobstore key
- # [
required] :api_token - user's API token
+ # [required] :
api_token - API token (signatures only work for this token)
+ # [
optional] :key - the Arvados server-side blobstore key
# [optional] :ttl - number of seconds before signature should expire
# [optional] :expire - unix timestamp when signature should expire
#
# [optional] :ttl - number of seconds before signature should expire
# [optional] :expire - unix timestamp when signature should expire
#
@@
-44,14
+44,16
@@
class Blob
end
timestamp = opts[:expire]
else
end
timestamp = opts[:expire]
else
- timestamp = db_current_time.to_i + (opts[:ttl] || 1209600)
+ timestamp = db_current_time.to_i +
+ (opts[:ttl] || Rails.configuration.blob_signature_ttl)
end
timestamp_hex = timestamp.to_s(16)
# => "53163cb4"
# Generate a signature.
signature =
end
timestamp_hex = timestamp.to_s(16)
# => "53163cb4"
# Generate a signature.
signature =
- generate_signature opts[:key], blob_hash, opts[:api_token], timestamp_hex
+ generate_signature((opts[:key] or Rails.configuration.blob_signing_key),
+ blob_hash, opts[:api_token], timestamp_hex)
blob_locator + '+A' + signature + '@' + timestamp_hex
end
blob_locator + '+A' + signature + '@' + timestamp_hex
end
@@
-88,7
+90,7
@@
class Blob
if !timestamp
raise Blob::InvalidSignatureError.new 'No signature provided.'
end
if !timestamp
raise Blob::InvalidSignatureError.new 'No signature provided.'
end
-
if !timestamp.match
/^[\da-f]+$/
+
unless timestamp =~
/^[\da-f]+$/
raise Blob::InvalidSignatureError.new 'Timestamp is not a base16 number.'
end
if timestamp.to_i(16) < (opts[:now] or db_current_time.to_i)
raise Blob::InvalidSignatureError.new 'Timestamp is not a base16 number.'
end
if timestamp.to_i(16) < (opts[:now] or db_current_time.to_i)
@@
-96,7
+98,8
@@
class Blob
end
my_signature =
end
my_signature =
- generate_signature opts[:key], blob_hash, opts[:api_token], timestamp
+ generate_signature((opts[:key] or Rails.configuration.blob_signing_key),
+ blob_hash, opts[:api_token], timestamp)
if my_signature != given_signature
raise Blob::InvalidSignatureError.new 'Signature is invalid.'
if my_signature != given_signature
raise Blob::InvalidSignatureError.new 'Signature is invalid.'