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

fix(wordlist): Removed redundant linejumps from CommonAdminBase64.txt #1142

Closed
wants to merge 1 commit into from

Conversation

ItsIgnacioPortal
Copy link
Collaborator

@ItsIgnacioPortal ItsIgnacioPortal commented Jan 2, 2025

Purpose of pull request

base64 strings in the 1st part of the list are built from strings without appended newline e.g. https://www.base64decode.org/dec/cm9vdA==

But the 2nd part strings got encoded wrongly with appending newline e.g. https://www.base64decode.org/dec/Um9vdDpSb290Cg==

I corrected all initial strings with awk trick so base64-ed strings have no newline encoded:
echo Root:Root | awk '{printf $1}' | base64

Additional context

The original PR is #1141. This PR fixes the commit message to comply with conventional commits.

EDIT: It's better to Squash-and-merge single-commit PRs, rather than making entirely new PRs.

@ItsIgnacioPortal ItsIgnacioPortal deleted the patch-1 branch January 2, 2025 23:08
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