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
Assignees
Milestone

Comments

@paragm
Copy link

@paragm 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

This comment has been minimized.

Copy link
Author

@paragm 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

This comment has been minimized.

Copy link
Author

@paragm paragm commented May 30, 2019

to clarify i am not behind proxy

@MRayermannMSFT MRayermannMSFT self-assigned this 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

This comment has been minimized.

Copy link
Member

@MRayermannMSFT MRayermannMSFT commented May 31, 2019

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

This comment has been minimized.

Copy link
Member

@MRayermannMSFT MRayermannMSFT commented Jun 3, 2019

Fix merged into master for 1.9

Storage Explorer automation moved this from Under Review to Done Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
2 participants
You can’t perform that action at this time.