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

Feature/819 add api option to get request response printed to log #820

Conversation

JoernBerkefeld
Copy link
Contributor

@JoernBerkefeld JoernBerkefeld commented Mar 27, 2023

PR details

What changes did you make? (Give an overview)

Is there anything you'd like reviewers to focus on?

...

Checklist

@JoernBerkefeld JoernBerkefeld added this to the 4.4.0 milestone Mar 27, 2023
@github-actions github-actions bot added the enhancement New feature or request; requires increasing the minor version of mcdev. Jira issue-type "Story" label Mar 27, 2023
@JoernBerkefeld JoernBerkefeld self-assigned this Mar 27, 2023
@JoernBerkefeld JoernBerkefeld added the e/IDE EPIC label Mar 27, 2023
'<fueloauth>*** TOKEN REMOVED ***</fueloauth>'
);
} else if (msg.headers?.Authorization) {
msg.headers.Authorization = 'Bearer *** TOKEN REMOVED ***';

Check failure

Code scanning / CodeQL

Hard-coded credentials

The hard-coded value "Bearer *** TOKEN REMOVED ***" is used as [authorization header](1).
@JoernBerkefeld JoernBerkefeld merged commit c371672 into develop Mar 27, 2023
@JoernBerkefeld JoernBerkefeld deleted the feature/819-add-api-option-to-get-request-response-printed-to-log branch March 27, 2023 22:21
@@ -469,23 +469,27 @@ const Util = {
if (argv.silent) {
// only errors printed to CLI
Util.logger.level = 'error';
Util.OPTIONS.loggerLevel = 'error';
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need this? or can we just set the line above?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the first one we need, the second one we technically don't. I figured it's good practice to have it all in one place though for reference by other classes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e/IDE EPIC enhancement New feature or request; requires increasing the minor version of mcdev. Jira issue-type "Story"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] add --api option to get request & response printed to log
2 participants