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

Improve provider name handling #213

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Commits on Jun 3, 2024

  1. Fix registration of dev provider in Service.authMiddleware.Providers

    Now it is possible to have a configuration,
    where only one single dev provider is enabled.
    
    Providers were not registered into Service.authMiddleware.Provicers slice
    in the Service.AddDevProvider() and Service.AddAppleProvider() methods before.
    cyb3r4nt committed Jun 3, 2024
    Configuration menu
    Copy the full SHA
    30656d6 View commit details
    Browse the repository at this point in the history

Commits on Jul 17, 2024

  1. Configuration menu
    Copy the full SHA
    ec4b929 View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2024

  1. Configuration menu
    Copy the full SHA
    ec38494 View commit details
    Browse the repository at this point in the history

Commits on Aug 29, 2024

  1. improve provider name handling

    Add provider name into JWT token claims
    to allow provider names with multiple underscore "_" symbols.
    Forbid provider names containing URL reserved symbols.
    cyb3r4nt committed Aug 29, 2024
    Configuration menu
    Copy the full SHA
    f13649e View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8fe2f59 View commit details
    Browse the repository at this point in the history

Commits on Sep 2, 2024

  1. fix race conditions in TestTelegramConfirmedRequest

    There were two different race conditions between logic in TestTelegramConfirmedRequest
    and TelegramAPIMock.GetUpdatesFunc and TelegramAPIMock.SendFunc:
    * GetUpdatesFunc may start before token was fetched,
    then it produces empty telegramUpdate response, which causes assertions in SendFunc to fail.
    * When token becomes used and removed from wait queue after successful login completion,
    then GetUpdatesFunc may be still called and new telegram update is created for same token.
    This breaks telegram update processing logic,
    and SendFunc gets called with the error parameter, which also breaks assertions.
    cyb3r4nt committed Sep 2, 2024
    Configuration menu
    Copy the full SHA
    178b002 View commit details
    Browse the repository at this point in the history
  2. fix golangci-lint warnings

    cyb3r4nt committed Sep 2, 2024
    Configuration menu
    Copy the full SHA
    0c9db77 View commit details
    Browse the repository at this point in the history
  3. 📖 docs: Update README

    tomy0000000 authored and cyb3r4nt committed Sep 2, 2024
    Configuration menu
    Copy the full SHA
    5a80324 View commit details
    Browse the repository at this point in the history