Skip to content

Conversation

@filipbodor
Copy link
Contributor

As we now use swift-log in slido for mac project, it makes sense to transfer to the logging system also in this swift library.

Removed:
I removed old logging system which was used by this library.

Implemented:
Instead of storing OAuth2Logger instance, the library will store Logger instance from swift-log framework.
I needed to change the way log messages were written:

logger.warn()                                 // -> logger.warning()
logger.info("OAuth2", msg: "message")         // -> logger.info("message")

@DominikPalo
Copy link
Member

DominikPalo commented Sep 8, 2025

@filipbodor could you please link this PR with #24 issue to make it trackable and update the status (and other details, like "assignee") of that issue? We use this approach for all other features in this library, since we don't use Jira for open-source projects.

@filipbodor filipbodor self-assigned this Sep 9, 2025
@filipbodor filipbodor linked an issue Sep 9, 2025 that may be closed by this pull request
@filipbodor filipbodor added the feature New feature or request label Sep 9, 2025
@filipbodor filipbodor force-pushed the feature/PS-23075-implement-swift-log branch from 038d087 to ec6371d Compare September 9, 2025 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Idea: Use the Swift Logging

2 participants