Skip to content
This repository has been archived by the owner on Oct 28, 2024. It is now read-only.

feat: allow to remove padding #1059

Merged
merged 1 commit into from
Mar 29, 2021
Merged

Conversation

kuisathaverat
Copy link
Contributor

What does this PR do?

It adds a parameter to disable base64 padding.

Why is it important?

When you disable padding the base64 string contains only a-z, A-Z, 0-9 characters.

@kuisathaverat kuisathaverat added enhancement New feature or request automation Team:Automation Label for the Observability productivity team labels Mar 29, 2021
@kuisathaverat kuisathaverat requested a review from a team March 29, 2021 15:12
@kuisathaverat kuisathaverat self-assigned this Mar 29, 2021
@botelastic botelastic bot added the groovy label Mar 29, 2021
@apmmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #1059 opened

  • Start Time: 2021-03-29T15:12:36.350+0000

  • Duration: 42 min 3 sec

  • Commit: 99e01dc

Test stats 🧪

Test Results
Failed 0
Passed 1088
Skipped 2
Total 1090

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 1088
Skipped 2
Total 1090

@kuisathaverat kuisathaverat merged commit 9fb949e into elastic:master Mar 29, 2021
v1v referenced this pull request in v1v/apm-pipeline-library Apr 6, 2021
* 'master' of github.com:elastic/apm-pipeline-library: (40 commits)
  Fix and simplify buildStatus step (#1066)
  Add benchmark reporter image to build pipeline (#1065)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.207
  feat: stage status cache (#1061)
  feat: step hasCommentAuthorWritePermissions (#1064)
  ARM workers are now ephemeral (#1062)
  docs: update metricbeat step docs (#1057)
  Update beats-ci workers for the test-infra validation (#1058)
  feat: allow to remove padding (#1059)
  mergify: priority is now a pro feature (#1060)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.206
  chore(deps): bump groovy-eclipse-batch from 3.0.7-02 to 3.0.7-03 (#1045)
  chore: bump Elastic Stack versions (#1052)
  Add GCE JCasC (#1053)
  feat: metricbeat step (#1050)
  chore: refactor filebeat step (#1049)
  ...
@kuisathaverat kuisathaverat deleted the no_padding branch July 19, 2021 14:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automation enhancement New feature or request groovy Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants