-
Notifications
You must be signed in to change notification settings - Fork 3
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: added pinojs and logger aggregation using asynclocalstorage implementation. [v.0] #65
base: develop
Are you sure you want to change the base?
Conversation
src/core/infrastructure/midaz/portfolios/midaz-create-portfolio-repository.ts
Outdated
Show resolved
Hide resolved
src/core/infrastructure/container-registry/container-registry.ts
Outdated
Show resolved
Hide resolved
src/app/api/organizations/[id]/ledgers/[ledgerId]/portfolios/[portfolioId]/route.ts
Outdated
Show resolved
Hide resolved
src/app/api/organizations/[id]/ledgers/[ledgerId]/portfolios/[portfolioId]/route.ts
Outdated
Show resolved
Hide resolved
src/app/api/organizations/[id]/ledgers/[ledgerId]/portfolios/[portfolioId]/route.ts
Outdated
Show resolved
Hide resolved
const { message, status } = await apiErrorHandler(error) | ||
return NextResponse.json(portfolio) | ||
} catch (error: any) { | ||
console.error('Error getting portfolio', error) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remover console.log()
body: JSON.stringify(portfolio) | ||
body: JSON.stringify(portfolio), | ||
headers: { | ||
'X-Midaz-Id': midazId |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O nome do header não é Midaz-Id? Segundo a documentação, seria esse o nome do atributo do header.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -11,6 +11,7 @@ export class MidazCreatePortfolioRepository | |||
async create( | |||
organizationId: string, | |||
ledgerId: string, | |||
midazId: string, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não esta enviando o midazId para o midaz, precisa incluir no header.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
src/lib/logger/request-context.ts
Outdated
...event | ||
} | ||
|
||
// switch (event.level) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remover comentários.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verificar comentários.
Midaz Console Pull Request Checklist
Pull Request Type
Checklist
Please check each item after it's completed.
Additional Notes
Obs: Please, always remember to target your PR to develop branch instead of main.