akamai / uls

Unified Log Streamer (ULS)
Apache License 2.0
30 stars 10 forks source link

[BUG][Minor] get-uls.sh requires pip 22.2 #58

Closed bitonio closed 7 months ago

bitonio commented 7 months ago

On a fresh Rocky 9, the get-uls.sh script spills out some pip error due to the argument --dry-run introduced at version 22.2.

# pip3 --version
pip 21.2.3 from /usr/lib/python3.9/site-packages/pip (python 3.9)

To simple solve the issue, upgrade pip with pip install --upgrade pip. There might be other option with the dnf package manager.

bitonio commented 7 months ago

I made some change to detect pip version and that should offer a better experience on old systems:

[root@172-XXX-XXX-XX ~]# bash get-uls.sh

       AKAMAI
   _/    _/  _/          _/_/_/
  _/    _/  _/        _/
 _/    _/  _/          _/_/
_/    _/  _/              _/
 _/_/    _/_/_/_/  _/_/_/

Unified Log Streamer - Installer

Available ULS modules:
  eaa: Enterprise Application Access
  etp: Secure Internet Access Enterprise
  mfa: Akamai phishproof MFA
  gc:  Akamai Guardicore Segmentation
  ln:  Linode Audit log

More about supported feed:
https://github.com/akamai/uls/blob/main/docs/LOG_OVERVIEW.md

pip3 version must be >= 22.2, version detected: 21.2.3
Consider upgrading your PIP version with command:
  /usr/bin/pip3 install --upgrade pip
MikeSchiessl commented 7 months ago

It is available in the current "development" branch and will be GA in v1.7.3

MikeSchiessl commented 7 months ago

fixed in Version 1.7.3