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

Remove apiclientlib #486

Merged
merged 2 commits into from
Jul 26, 2024
Merged

Remove apiclientlib #486

merged 2 commits into from
Jul 26, 2024

Conversation

protoism
Copy link
Contributor

plugin apiclientlib is used only for percent encoding.

I replaced google-http-java-client's
PercentEncoder with java's URLEncoder

So... apiclientlib can be removed

@protoism protoism changed the base branch from main to feature/fix_eclipse_compilation July 21, 2024 16:54
@protoism protoism requested review from niloc132 and foal July 21, 2024 17:06
@niloc132
Copy link
Member

I think you meant to target this to main?

@niloc132 niloc132 mentioned this pull request Jul 21, 2024
@@ -169,7 +168,7 @@ private static String getParametersString(Map<String, String> parametersMap) {
}
resultBuilder.append(entry.getKey());
resultBuilder.append('=');
resultBuilder.append(ESCAPER.escape(entry.getValue()));
resultBuilder.append(URLEncoder.encode(entry.getValue(), Charsets.UTF_8));
Copy link
Member

Choose a reason for hiding this comment

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

could also be java.nio.charset.StandardCharsets.UTF_8

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes!

@protoism
Copy link
Contributor Author

I think you meant to target this to main?
After merging feature/fix_eclipse_compilation I'll rebase it.
3 PRs one on top of each other.
Simplest way to test all of them together

Base automatically changed from feature/fix_eclipse_compilation to main July 26, 2024 00:53
@protoism protoism force-pushed the feature/remove_apiclientlib branch from b000046 to 3e95247 Compare July 26, 2024 01:05
@protoism protoism merged commit 69ff577 into main Jul 26, 2024
2 checks passed
@protoism protoism deleted the feature/remove_apiclientlib branch July 26, 2024 01:31
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