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

feat: middleware logger better logger scopes #7281

Merged
merged 8 commits into from
Oct 21, 2024

Conversation

Fifciu
Copy link
Collaborator

@Fifciu Fifciu commented Oct 14, 2024

@Fifciu Fifciu requested a review from a team as a code owner October 14, 2024 13:44
Copy link

changeset-bot bot commented Oct 14, 2024

🦋 Changeset detected

Latest commit: a78fa3e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@vue-storefront/middleware Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Fifciu Fifciu force-pushed the feat/middleware-logger-better-logger-scopes branch from 6bdd93a to fbd42e5 Compare October 14, 2024 13:47
@Fifciu Fifciu marked this pull request as draft October 14, 2024 14:27
@Fifciu Fifciu force-pushed the feat/middleware-logger-better-logger-scopes branch from fbd42e5 to d56c6b1 Compare October 15, 2024 14:23
@Fifciu Fifciu force-pushed the feat/middleware-logger-better-logger-scopes branch 2 times, most recently from 21f93b7 to 61705f3 Compare October 17, 2024 16:40
packages/middleware/src/apiClientFactory/index.ts Outdated Show resolved Hide resolved
packages/middleware/src/apiClientFactory/index.ts Outdated Show resolved Hide resolved
packages/middleware/src/apiClientFactory/index.ts Outdated Show resolved Hide resolved
packages/middleware/src/apiClientFactory/index.ts Outdated Show resolved Hide resolved
@Fifciu Fifciu force-pushed the feat/middleware-logger-better-logger-scopes branch from 61705f3 to f808e08 Compare October 18, 2024 09:57
@Fifciu Fifciu force-pushed the feat/middleware-logger-better-logger-scopes branch from f808e08 to 227be66 Compare October 18, 2024 11:56
@Fifciu Fifciu force-pushed the feat/middleware-logger-better-logger-scopes branch from 227be66 to 62c10ed Compare October 18, 2024 12:55
@Fifciu Fifciu force-pushed the feat/middleware-logger-better-logger-scopes branch from b4afc70 to 180d238 Compare October 21, 2024 09:25
@Fifciu Fifciu marked this pull request as ready for review October 21, 2024 12:14
@Fifciu Fifciu merged commit dcb8fa7 into feat/logger Oct 21, 2024
9 checks passed
@Fifciu Fifciu deleted the feat/middleware-logger-better-logger-scopes branch October 21, 2024 12:16
bartoszherba added a commit that referenced this pull request Nov 7, 2024
* feat: logger 1.0.0-rc0

* chore: add changeset

* ci: release (rc) (#7266)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger rc.1

* ci: release (rc) (#7268)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger 1.0.0-rc.2

* ci: release (rc) (#7272)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: add logger module to the SDK (#7267)

* ci: release (rc) (#7273)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger rc.3

* feat: rc.3

* ci: release (rc) (#7277)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: middleware logger (#7269)

* feat: injecting logger and metadata

* feat: injecting logger in hooks, onCreate, init fn, removed consola

* refactor: tidying, adding and adjusting tests

* refactor: applied CR tips

* chore: fix sdk test

* refactor: implemented CR suggestions

---------

Co-authored-by: Bartosz Herba <[email protected]>

* ci: release (rc) (#7278)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: add "class-methods-use-this": "off" rule (#7274)

* ci: release (#7275)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* CLOUD-3578 Improve Kubernetes Probes docs (#7264)

* CLOUD-3578 Improve readiness probe docs

* Update docs/content/3.middleware/2.guides/9.kubernetes-probes.md

Co-authored-by: Matt Maribojoc <[email protected]>

* Add changes from Kuba

---------

Co-authored-by: Matt Maribojoc <[email protected]>

* IN-3956 - Middleware extension docs is missing isNamespaced in the example code (#7271)

Co-authored-by: krystian wlodarski <[email protected]>

* feat: sdk logger wrapper (#7276)

* chore: update changeset

* ci: release (rc) (#7279)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: add CR fixes

* feat: middleware logger better logger scopes (#7281)

* fix: moved global logger error to non overwritable part of code

* chore: snapshot of done work

* refactor: MethodDecoratorManager

* refactor: reduced usage of singleton to support parallel calls of fns

* fix: infinite loop constraint on types

* Revert "fix: infinite loop constraint on types"

This reverts commit 01904e3.

* chore: bumped ts jest, and jest

* fix: test

* ci: release (rc) (#7284)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* docs: working with logger (#7285)

* fix: merging logger configs (#7286)

* docs(tfal-56): next/nuxt logger implementation (#7282)

* docs: update next/nuxt docs (#7290)

* chore: export types from logger module (#7292)

* ci: release (rc) (#7293)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger middleware error boundary (#7291)

* feat: error boundary

* refactor: refactor

* feat: use appropriate verbosity levels specifically in the browser environment (#7296)

* ci: release (rc) (#7297)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: remove logger module (#7298)

* feat: block overwriting alokai metadata (#7300)

* feat: block overwriting alokai metadata

* refactor: outsourced deep merging of obj's to lodash.merge

* feat: guard to prevent overwriting alokai key in metadata obj

* fix: tests

* Update packages/middleware/src/logger/injectMetadata.ts

Co-authored-by: Bartosz Herba <[email protected]>

---------

Co-authored-by: Bartosz Herba <[email protected]>

* ci: release (rc) (#7303)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: replace console with alokai logger in the middleware module (#7299)

* feat: change log level to verbosity (#7306)

* ci: release (rc) (#7307)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Bartosz Herba <[email protected]>

* docs(tfal-89,tfal-83): update logger docs for current solution (#7308)

* docs(tfal-89,tfal-83): update logger docs for current solution

* docs: fix

* docs: alokai metadata (#7305)

* docs: block overriding alokai key in provided metadata

* Apply suggestions from code review

Co-authored-by: daaf <[email protected]>

* Apply suggestions from code review

---------

Co-authored-by: daaf <[email protected]>
Co-authored-by: Bartosz Herba <[email protected]>

* chore: remove custom handler docs

* chore: exit pre mode

* feat: update logger exports and docs

* chore: update docs after cr

* docs: fix sdk docs

* feat: change log structured interface parameters

* Apply suggestions from code review - missing type annotation

Co-authored-by: daaf <[email protected]>

* chore: update node env for logger unit tests

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: daaf <[email protected]>
Co-authored-by: Filip Jędrasik <[email protected]>
Co-authored-by: Artur <[email protected]>
Co-authored-by: Matt Maribojoc <[email protected]>
Co-authored-by: Krystian Wlodarski <[email protected]>
Co-authored-by: krystian wlodarski <[email protected]>
Co-authored-by: Dawid Ziobro <[email protected]>
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.

2 participants