caronc / apprise

Apprise - Push Notifications that work with just about every platform!
https://hub.docker.com/r/caronc/apprise
BSD 2-Clause "Simplified" License
10.9k stars 384 forks source link

clicksend:// authentication bugfix #1121

Closed caronc closed 1 month ago

caronc commented 1 month ago

Description:

Related issue (if applicable): #1103

Previous to this merge request, clicksend://user:apikey did not work correctly

Checklist

Testing

Anyone can help test this source code as follows:

# Create a virtual environment to work in as follows:
python3 -m venv apprise

# Change into our new directory
cd apprise

# Activate our virtual environment
source bin/activate

# Install the branch
pip install git+https://github.com/caronc/apprise.git@1103-clicksend-authfix

# Test out the changes with the following command:
apprise -t "Test Title" -b "Test Message" \
  "clicksend://user:apikey@..."
codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.28%. Comparing base (fcd70ff) to head (006c5fe).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1121 +/- ## ======================================= Coverage 99.28% 99.28% ======================================= Files 139 139 Lines 18066 18067 +1 Branches 3689 3690 +1 ======================================= + Hits 17936 17937 +1 Misses 121 121 Partials 9 9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.