Thomas Kishel

I don't have enough of an understanding of the potential impact of this change to the internal workflows of the project (aka: I know enough to be dangerous ) and...

I'm open to any implementation that allows users to configure querying status independently from controlling the server :) Might you have an example that illustrates what you are thinking about?

How about this ... config ``` control-url "ssl://0.0.0.0:12345? ..." control-token "hello" control-actions ['stats'] ``` status.rb ``` def initialize(cli, token = nil, actions = []) @cli = cli @auth_token = token...

See https://github.com/puma/puma/pull/2083 Not a lambda, but very simple, and I assume the array could be replaced ...

It is, but ... 1. Having the token provide (remote) access to both control and status in the default app is an inherent security issue. 2. Empirically (this is where...

Yes: it would be lovely to not need a token to access the `stats` and `gc_stats` endpoints. > No, it just means we're over-securing status. So, should I close both...

Yay! I remember seeing that but did not make the connection.

If `cwp` is specified as a parameter: https://github.com/PaloAltoNetworks/pcs-sizing-scripts/blob/main/aws/resource-count-aws.sh#L56 We do count EKS clusters: https://github.com/PaloAltoNetworks/pcs-sizing-scripts/blob/main/aws/resource-count-aws.sh#L528 https://github.com/PaloAltoNetworks/pcs-sizing-scripts/blob/main/aws/resource-count-aws.sh#L259 https://github.com/PaloAltoNetworks/pcs-sizing-scripts/blob/main/aws/resource-count-aws.sh#L498 And we always count ELBs: https://github.com/PaloAltoNetworks/pcs-sizing-scripts/blob/main/aws/resource-count-aws.sh#L259 You can verify this by adding `-x` to...

This mifght be solved via one of these answers: https://stackoverflow.com/questions/14742064/python-os-environhome-works-on-idle-but-not-in-a-script Replace `os.environ['HOME']` with `os.path.expanduser('~')` Or `set HOME=%USERPROFILE%` Feel free to submit a PR for the former, and note that this...