Skip to content

RNGP - Sanitize the output of the config command#46482

Closed
cortinico wants to merge 1 commit into
facebook:mainfrom
cortinico:export-D62641979
Closed

RNGP - Sanitize the output of the config command#46482
cortinico wants to merge 1 commit into
facebook:mainfrom
cortinico:export-D62641979

Conversation

@cortinico
Copy link
Copy Markdown
Contributor

Summary:
Fixes #46443
Fixes #46134

I'm sanitizing the output of the config command + I've added some more logging in case of failure.

Changelog:
[Android] [Fixed] - RNGP - Sanitize the output of the config command

Differential Revision: D62641979

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Sep 13, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D62641979

Summary:
Pull Request resolved: facebook#46482

Fixes facebook#46443
Fixes facebook#46134

I'm sanitizing the output of the `config` command + I've added some more logging in case of failure.

Changelog:
[Android] [Fixed] - RNGP - Sanitize the output of the config command

Reviewed By: cipolleschi

Differential Revision: D62641979
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D62641979

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in d7884a6.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 13, 2024
@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @cortinico in d7884a6

When will my fix make it into a release? | How to file a pick request?

cipolleschi pushed a commit that referenced this pull request Sep 16, 2024
Summary:
Pull Request resolved: #46482

Fixes #46443
Fixes #46134

I'm sanitizing the output of the `config` command + I've added some more logging in case of failure.

Changelog:
[Android] [Fixed] - RNGP - Sanitize the output of the config command

Reviewed By: cipolleschi

Differential Revision: D62641979

fbshipit-source-id: c13d27a42beeb7a973c1802e7204631d49d3d09b
blakef pushed a commit that referenced this pull request Sep 30, 2024
Summary:
Pull Request resolved: #46482

Fixes #46443
Fixes #46134

I'm sanitizing the output of the `config` command + I've added some more logging in case of failure.

Changelog:
[Android] [Fixed] - RNGP - Sanitize the output of the config command

Reviewed By: cipolleschi

Differential Revision: D62641979

fbshipit-source-id: c13d27a42beeb7a973c1802e7204631d49d3d09b
@blakef blakef mentioned this pull request Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

3 participants