Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to load subscriptions when account has access to more than 50 #1416

Closed
paragm opened this issue May 28, 2019 · 4 comments
Closed

Unable to load subscriptions when account has access to more than 50 #1416

paragm opened this issue May 28, 2019 · 4 comments
Assignees
Labels
🪲 bug Issue is not intended behavior ✅ merged A fix for this issue has been merged
Milestone

Comments

@paragm
Copy link

paragm commented May 28, 2019

Storage Explorer Version: 1.8.1
Platform/OS: macOS mojave
Architecture: x64
Regression From: 1.7.0
Bug description
while running 1.7.0 we had about 30 subscriptions that were getting loaded just fine. between 1.7.1 and 1.8.1 something has changed on the storage explorer side for sure. we currently have 100+ subscription IDs under single tenant and i am the co-admin/owner on all of them.
Steps to Reproduce

  1. have tenant id with more than 100+ subscriptions
  2. have fresh install of osx mojave and fresh install of storage explorer 1.7.1 with upgrade to 1.8.1
  3. now add azure account and see if subscriptions load
  4. it is able to load the second tenant my account has access to but primary tenant id it constantly gets the same auth error. it kind of seems like an incorrect message being shown as single account having access to multiple tenant ids and one loads but the other gets auth error..

Expected Experience
expected behavior to load all subscriptions in the client

Actual Experience
getting this error

Subscriptions for the follwowing tenants could not be retrieved:
Tenant Id: 9421890a-cf2d-4b02-b928-e6ab9894b909, Error: {"message":"HTTP ERROR 401: Unauthorized","response":{"statusCode":401,"body":"{\"error\":{\"code\":\"ExpiredAuthenticationToken\",\"message\":\"The access token expiry UTC time '5/27/2019 5:57:03 AM' is earlier than current UTC time '5/27/2019 5:57:03 AM'.\"}}","headers":{"cache-control":"no-cache","pragma":"no-cache","content-type":"application/json; charset=utf-8","expires":"-1","www-authenticate":"Bearer authorization_uri=\"https://login.windows.net/\", error=\"invalid_token\", error_description=\"The access token has expired.\"","x-ms-failure-cause":"gateway","x-ms-request-id":"825a68da-1935-44cc-afef-ae643ba83799","x-ms-correlation-request-id":"825a68da-1935-44cc-afef-ae643ba83799","x-ms-routing-request-id":"CANADACENTRAL:20190527T055703Z:825a68da-1935-44cc-afef-ae643ba83799","strict-transport-security":"max-age=31536000; includeSubDomains","x-content-type-options":"nosniff","date":"Mon, 27 May 2019 05:57:03 GMT","connection":"close","content-length":"172"},"request":{"uri":{"protocol":"https:","slashes":true,"auth":null,"host":"management.azure.com","port":443,"hostname":"management.azure.com","hash":null,"search":"?api-version=2015-01-01","query":"api-version=2015-01-01","pathname":"//subscriptions","path":"//subscriptions?api-version=2015-01-01","href":"https://management.azure.com//subscriptions?api-version=2015-01-01"},"method":"GET","headers":{"Content-Type":"application/json","Authorization":"Bearer eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsIng1dCI6IkhCeGw5bUFlNmd4YXZDa2NvT1UyVEhzRE5hMCIsImtpZCI6IkhCeGw5bUFlNmd4YXZDa2NvT1UyVEhzRE5hMCJ9.eyJhdWQiOiJodHRwczovL21hbmFnZW1lbnQuY29yZS53aW5kb3dzLm5ldC8iLCJpc3MiOiJodHRwczovL3N0cy53aW5kb3dzLm5ldC85NDIxODkwYS1jZjJkLTRiMDItYjkyOC1lNmFiOTg5NGI5MDkvIiwiaWF0IjoxNTU4OTMyNjkzLCJuYmYiOjE1NTg5MzI2OTMsImV4cCI6MTU1ODkzNjU5MywiYWNyIjoiMSIsImFpbyI6IjQyWmdZTmduNHUyM3JLUmRjOC9XckxPdXQ4dXluMDVhNXFCbkh2YWcrWXZyQ1RjSkMwY0EiLCJhbXIiOlsicHdkIl0sImFwcGlkIjoiMDRiMDc3OTUtOGRkYi00NjFhLWJiZWUtMDJmOWUxYmY3YjQ2IiwiYXBwaWRhY3IiOiIwIiwiZmFtaWx5X25hbWUiOiJNZWh0YSIsImdpdmVuX25hbWUiOiJQYXJhZyIsImdyb3VwcyI6WyJlMzYyZjdlNS1lNDhlLTQzYzctYmIwNy1mYzMxZjU1OWY0NzciLCJhOTczMmQ0MC1iMWZiLTQwNDYtYTJlMS04NmFkY2IyZmZiYTkiLCJlNTI4YTBlNi0xNzM5LTQ0NWMtOTE3ZS01ZmJhOWQ5ZWViMDYiLCJkZjg1MTE3ZS0xOGI1LTQzZDUtYThiMS00MDg1MWY4ODhkZDYiLCI2NTQ1ZjUzZi04OGNkLTRhNWItOGJjMy1kMTZiYzY2NWJjOTciXSwiaXBhZGRyIjoiNzMuOTMuNDEuMjA1IiwibmFtZSI6InBhcmFnIiwib2lkIjoiZTMwMzI5YTUtNmQ4ZS00ODBiLTlmNDEtOTliMjRjNDVlN2IwIiwicHVpZCI6IjEwMDMwMDAwQTVENDk3REEiLCJwd2RfZXhwIjoiOTEzOTg4IiwicHdkX3VybCI6Imh0dHBzOi8vcG9ydGFsLm1pY3Jvc29mdG9ubGluZS5jb20vQ2hhbmdlUGFzc3dvcmQuYXNweCIsInNjcCI6InVzZXJfaW1wZXJzb25hdGlvbiIsInN1YiI6ImU5NkpZTTRIQmhvQWlYTGFTMDNhMUpjRERoSHdoM3BCU1ppRmNka1FxVVEiLCJ0aWQiOiI5NDIxODkwYS1jZjJkLTRiMDItYjkyOC1lNmFiOTg5NGI5MDkiLCJ1bmlxdWVfbmFtZSI6InBhcmFnQGNsb3Vkc2ltcGxlLmNvbSIsInVwbiI6InBhcmFnQGNsb3Vkc2ltcGxlLmNvbSIsInV0aSI6IlBCR0VBZ1UxU1VHUmVqd3BEV2xNQUEiLCJ2ZXIiOiIxLjAiLCJ3aWRzIjpbIjYyZTkwMzk0LTY5ZjUtNDIzNy05MTkwLTAxMjE3NzE0NWUxMCIsIjE3MzE1Nzk3LTEwMmQtNDBiNC05M2UwLTQzMjA2MmNhY2ExOCIsImIwZjU0NjYxLTJkNzQtNGM1MC1hZmEzLTFlYzgwM2YxMmVmZSJdfQ.aAG1det4hAqoHZ4VFRvxMXY38-4fx9M7l0c34KBXuDdrwoMVMFMFdV6lbzIazThxxEbXTlJn2yajXQkefZNOxnthG8_tWedOERnLBFqtnclolBbFoGZtkpzrcBjaOom3KmNi4VAIwzXj2LDkcYeDfDWXeUKq7LwI2CEesZ6GVl9plCZSEc7NDgfx0KxtAF97g71FtVnU0vaa9SHUJ_eI1ZC9f4LnX3-0bgDlfYsbo2PO92AE5mPTEnN_i0DhyadeXCM-FguXcTiS8SBly0vacng0LZ6j69TMkL9WeV1-Pmjfa3dW63ItBfopBX32PFRnKVMOSP34YxiAb9O0tNoG0w"}}},"body":"{\"error\":{\"code\":\"ExpiredAuthenticationToken\",\"message\":\"The access token expiry UTC time '5/27/2019 5:57:03 AM' is earlier than current UTC time '5/27/2019 5:57:03 AM'.\"}}"}

Additional Context
more than happy to run additional debug steps

@paragm
Copy link
Author

paragm commented May 30, 2019

after doing reset and reload from the menu options i am not getting the following error message:

Subscriptions for the follwowing tenants could not be retrieved:
Tenant Id: 9421890a-cf2d-4b02-b928-e6ab9894b909, Error: {"code":"ECONNRESET","errno":"ECONNRESET","syscall":"read"}

@paragm
Copy link
Author

paragm commented May 30, 2019

to clarify i am not behind proxy

@MRayermannMSFT MRayermannMSFT self-assigned this May 31, 2019
@MRayermannMSFT MRayermannMSFT added the 🪲 bug Issue is not intended behavior label May 31, 2019
@MRayermannMSFT MRayermannMSFT added this to the 1.9.0 milestone May 31, 2019
@MRayermannMSFT MRayermannMSFT added this to Committed in Storage Explorer via automation May 31, 2019
@MRayermannMSFT MRayermannMSFT changed the title constantly getting unable to load all subscriptions Unable to load subscriptions when account has access to more than 50 May 31, 2019
@MRayermannMSFT
Copy link
Member

We aren't handling nextLink right in _listSubscriptions, I'll fix it!

@MRayermannMSFT MRayermannMSFT moved this from Committed to In Progress in Storage Explorer May 31, 2019
@MRayermannMSFT MRayermannMSFT moved this from In Progress to Under Review in Storage Explorer Jun 3, 2019
@MRayermannMSFT
Copy link
Member

Fix merged into master for 1.9

Storage Explorer automation moved this from Under Review to Done Jun 3, 2019
@MRayermannMSFT MRayermannMSFT added the ✅ merged A fix for this issue has been merged label Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 bug Issue is not intended behavior ✅ merged A fix for this issue has been merged
Projects
Development

No branches or pull requests

2 participants