Slack now uses a different keychain identifier for storing its password on macOS. This PR simply updates to that new identifier.
Unfortunately we don't have control or visibility on potentially breaking changes in Slack because we're not using a published API. So this is a reactive change and required reverse engineering. I've also made the assumption that Slack's change has been rolled out widely enough that there's no need to be backward compatible. The assumption is based on the fact that Slack requires frequent updates to be compatible with the backend, so older versions are unlikely to be functional anyway.
Status
READY - tested from fresh installation on two macs, and used occasionally since the issue was raised 4 months ago (and was probably present for some time before that).
Description
Fixes https://github.com/n8henrie/pycookiecheat/issues/54
Slack now uses a different keychain identifier for storing its password on macOS. This PR simply updates to that new identifier.
Unfortunately we don't have control or visibility on potentially breaking changes in Slack because we're not using a published API. So this is a reactive change and required reverse engineering. I've also made the assumption that Slack's change has been rolled out widely enough that there's no need to be backward compatible. The assumption is based on the fact that Slack requires frequent updates to be compatible with the backend, so older versions are unlikely to be functional anyway.
Status
READY - tested from fresh installation on two macs, and used occasionally since the issue was raised 4 months ago (and was probably present for some time before that).
Related Issues
Todos
Steps to Test or Reproduce
As per the original PR: