ManageIQ / azure-armrest

Ruby interface for Azure using the new REST API
Apache License 2.0
15 stars 36 forks source link

next_marker_results is busted #335

Closed djberg96 closed 6 years ago

djberg96 commented 7 years ago

There's some kind of internal bug with regards to handling of nextmarker results. To duplicate, you can try this:

sas = Azure::Armrest::StorageAccountService.new(conf)

# Pick a storage account that has a share with 2 or more files.
storage_acct_name = 'foo'
resource_group  = 'bar'

acct = sas.get(storage_acct_name, resource_group)

keys = sas.list_account_keys(acct.name, acct.resource_group)
key  = keys['key1'] || keys['key2']

files = acct.files(your_share, key, :maxresults => 1)

Inspection reveals that the nextmarker is being appended to the base URL instead of retained as part of the query.

djberg96 commented 6 years ago

Fixed by https://github.com/ManageIQ/azure-armrest/pull/336