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

Upgrade logger and add fallback calculation #8145

Closed
wants to merge 62 commits into from

Conversation

PineappleIOnic
Copy link
Member

What does this PR do?

This PR upgrades the logger and adds a fallback calculation for older env-var types as to not break compatibility.

Test Plan

Test with both new and old type, both work

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

Akhil Anand and others added 30 commits October 5, 2023 16:22
This workflow action uses OSV Scanner, an open source vulnerability
scanner by Google. We're using OSV Scanner because it has:

* good usability - JSON output and multiple options
* good accuracy - OSV database from google and support
  for multiple languages including PHP
Due to failures, changing to not_planned
Add OSV Scanner for vulnerability scans
chore: fix some typos in comments
…rying-deployments

Fix: Use `resourceInternalId` for Querying Function Deployments
…adme

Replace missing domain paths in README.md
Prevent functions domain to be used as custom domain
Meldiron and others added 27 commits May 7, 2024 09:05
…-from-censosud-and-consosud-to-cencosud

fix: updated typo in credit-cards.php credit card image filename
Co-authored-by: Christy Jacob <[email protected]>
Co-authored-by: Christy Jacob <[email protected]>
Co-authored-by: Christy Jacob <[email protected]>
Co-authored-by: Christy Jacob <[email protected]>
Co-authored-by: Christy Jacob <[email protected]>
Co-authored-by: Christy Jacob <[email protected]>
Co-authored-by: Christy Jacob <[email protected]>
…n-warning-at-appwrite-doctor

Fix Deprecated Warning in Doctor.php
…ocale-endpoint-Palestine

Add new country Palestine with its translations
Improve container naming conventions
@@ -732,6 +732,16 @@ function (mixed $value) {
throw new Exception(Exception::GENERAL_SERVER_ERROR, "Logging provider not supported. Logging is disabled");
}

// Old Sentry Format conversion
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// Old Sentry Format conversion
// Old Sentry Format conversion. Fallback until the old syntax is completely deprecated.

@PineappleIOnic PineappleIOnic changed the base branch from main to refactor-usage-sn May 17, 2024 08:55
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.

None yet