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

SLAS shortcode and tenant options switched #265

Open
drewzboto opened this issue Nov 23, 2021 · 4 comments
Open

SLAS shortcode and tenant options switched #265

drewzboto opened this issue Nov 23, 2021 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@drewzboto
Copy link

drewzboto commented Nov 23, 2021

I was trying to use the new slas commands, but I believe on v2.8.0 the options are accidentally switched. It looks like sfcc-ci is picking up the tenant from the shortcode option and the shortcode from the tenant option

sfcc-ci slas:client:list -help
Usage: slas:client:list [options]

Lists all SLAS clients that belong to a given tenant

Options:
  --tenant <tenant>        the tenant id used for slas
  --shortcode <shortcode>  the organizations short code
  -j, --json               Formats the output in json
  -h, --help               output usage information

Steps to reproduce:

  1. Auth sfcc-ci
  2. Attempt to list clients: sfcc-ci slas:client:list --shortcode kv7kzm78 --tenant zzrf_001

Expected results:
List of SLAS clients is displayed for tenant zzrf_001 for shortcode kv7kzm78

Actual results:
Error is displayed:
Could not get tenants: request to https://zzrf_001.api.commercecloud.salesforce.com/shopper/auth-admin/v1/tenants/kv7kzm78/clients failed, reason: getaddrinfo ENOTFOUND zzrf_001.api.commercecloud.salesforce.com

However, if you flip the tenant and shortcode it works:
sfcc-ci slas:client:list --tenant kv7kzm78 --shortcode zzrf_001

@drewzboto
Copy link
Author

@hnestmann can you reproduce this?

@drewzboto
Copy link
Author

Related... i'm not sure if this is working as expected for listing tenants:

 ~/code/ sfcc-ci slas:tenant:list -help
Usage: slas:tenant:list [options]

Lists all tenants that belong to a given organization

Options:
  --shortcode <shortcode>  the organizations short code
  -j, --json               Formats the output in json
  -h, --help               output usage information

 ~/code/ sfcc-ci slas:tenant:list --shortcode kv7kzm78
Could not get tenants: tenantId is not defined

@hnestmann
Copy link
Collaborator

Hi Drew, I heard from a partner the same thing- Thanks for raising - we'l have a look

@hnestmann hnestmann self-assigned this Nov 24, 2021
@tobiaslohr tobiaslohr added the bug Something isn't working label Jan 13, 2022
@johnboxall
Copy link
Contributor

johnboxall commented Jan 28, 2022

WIP here: master...johnboxall:fix-slas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants