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

Update keyring username for Slack. #69

Merged
merged 1 commit into from
Sep 5, 2024
Merged

Conversation

hraftery
Copy link
Contributor

@hraftery hraftery commented Sep 5, 2024

Description

Just chasing the ever-changing internal details of Slack. Not sure when this change happened, it may have been quite some time ago, but it is present in version 4.39.95 of Slack for Mac.

hraftery added a commit to hraftery/slacktokens that referenced this pull request Sep 5, 2024
Another day, another change to Slack. New leveldb location found using "lsof". Also requires an update to pycookiecheat because the keyring name has also changed. See n8henrie/pycookiecheat#69.

All the docs for updating a PyPI package have changed since the last update, so also took the opportunity to get up to speed. It was awful, but seems to be working now. Basically setup.cfg is superseded and now everything goes in pyproject.toml.
@n8henrie n8henrie changed the base branch from master to dev September 5, 2024 12:21
@n8henrie
Copy link
Owner

n8henrie commented Sep 5, 2024

@hraftery so it sounds like you've tested this (and it's working) locally?

@n8henrie
Copy link
Owner

n8henrie commented Sep 5, 2024

(Also, thanks for the PR!)

@hraftery
Copy link
Contributor Author

hraftery commented Sep 5, 2024

so it sounds like you've tested this (and it's working) locally?

Correct, yes.

@n8henrie
Copy link
Owner

n8henrie commented Sep 5, 2024

Great, thanks!

@n8henrie n8henrie merged commit b48c134 into n8henrie:dev Sep 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants