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

Add two new flags to enable HTTP Server and clients logs #166

Merged
merged 3 commits into from
Jul 4, 2024

Conversation

jvanecek
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Dec 17, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.61%. Comparing base (d4eef66) to head (5ccbc8e).

Additional details and impacted files
@@                Coverage Diff                 @@
##           release-candidate     #166   +/-   ##
==================================================
  Coverage              99.61%   99.61%           
==================================================
  Files                    166      166           
  Lines                  13457    13539   +82     
==================================================
+ Hits                   13405    13487   +82     
  Misses                    52       52           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jvanecek jvanecek changed the title WIP: Enable to log all http requests Add two new flags to enable HTTP Server and clients logs Dec 17, 2022
@jvanecek jvanecek self-assigned this Dec 17, 2022
mtabacman
mtabacman previously approved these changes Dec 19, 2022
Copy link
Member

@gcotelli gcotelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jvanecek take a look at the comments.

source/Stargate-API-Skeleton/StargateApplication.class.st Outdated Show resolved Hide resolved
source/Stargate-Model/NeoJSONWriter.extension.st Outdated Show resolved Hide resolved
source/Stargate-Model/NeoJSONWriter.extension.st Outdated Show resolved Hide resolved
source/Stargate-Model/NeoJSONWriter.extension.st Outdated Show resolved Hide resolved
source/Stargate-Model/NeoJSONWriter.extension.st Outdated Show resolved Hide resolved
source/Stargate-Model/NeoJSONWriter.extension.st Outdated Show resolved Hide resolved
source/Stargate-API-Skeleton/StargateApplication.class.st Outdated Show resolved Hide resolved
Copy link
Member

@gcotelli gcotelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make the commit with Pharo 12 to avoid an huge list of cosmetic changes

rowan/projects/Hyperspace.ston Outdated Show resolved Hide resolved
source/BaselineOfStargate/BaselineOfStargate.class.st Outdated Show resolved Hide resolved
source/BaselineOfStargate/BaselineOfStargate.class.st Outdated Show resolved Hide resolved
@jvanecek jvanecek changed the title Add two new flags to enable HTTP Server and clients logs Draft: Add two new flags to enable HTTP Server and clients logs Jul 2, 2024
@jvanecek jvanecek changed the title Draft: Add two new flags to enable HTTP Server and clients logs Add two new flags to enable HTTP Server and clients logs Jul 3, 2024
mtabacman
mtabacman previously approved these changes Jul 3, 2024
… incoming requests logging parameterization

this is because the StargateApplication is an HTTP Server and doesn't make sense to include the outgoing logging parameterization since it will not be necessary used.
@gcotelli gcotelli merged commit fa225f6 into release-candidate Jul 4, 2024
114 checks passed
@gcotelli gcotelli deleted the zinc-logging branch July 4, 2024 13:46
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.

4 participants