Skip to content

Conversation

alterstep
Copy link
Contributor

Summary

Add documentation for the new encrypt_ip and decrypt_ip VRL functions that implement format-preserving encryption for IP addresses.

These functions support two modes:

  • AES128: Scrambles entire IP address using AES-128 encryption
  • PFX: Prefix-preserving mode that maintains network hierarchy

How did you test this PR?

I build the website and reviewed the output.

Change Type

  • Bug fix
  • New feature
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

Related VRL PR: vectordotdev/vrl#1506

@alterstep alterstep requested review from a team as code owners September 12, 2025 20:22
@github-actions github-actions bot added the domain: external docs Anything related to Vector's external, public documentation label Sep 12, 2025
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Sep 15, 2025
Copy link
Member

@pront pront left a comment

Choose a reason for hiding this comment

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

Some words need to be added to allow.txt

@alterstep alterstep force-pushed the ipcrypt branch 2 times, most recently from b0ab89e to b38b78d Compare September 16, 2025 17:56
@pront pront enabled auto-merge September 16, 2025 17:56
auto-merge was automatically disabled September 16, 2025 18:47

Head branch was pushed to by a user without write access

@pront pront enabled auto-merge September 16, 2025 19:07
@alterstep
Copy link
Contributor Author

Would you like me to commit the regenerated licenses file?
It introduces many unrelated changes

@pront
Copy link
Member

pront commented Sep 16, 2025

Would you like me to commit the regenerated licenses file? It introduces many unrelated changes

Yes please, I tried to do it but my push was rejected. We want this PR to go in before the next release.

auto-merge was automatically disabled September 16, 2025 20:58

Head branch was pushed to by a user without write access

@pront pront enabled auto-merge September 16, 2025 21:06
@alterstep
Copy link
Contributor Author

I ran cargo vdev build licenses and make check-licenses, and both passed on my end. Did I miss something?

Does it need a rebase?

Add documentation for the new `encrypt_ip` and `decrypt_ip` VRL functions
that implement format-preserving encryption for IP addresses.

These functions support two modes:
- AES128: Scrambles entire IP address using AES-128 encryption
- PFX: Prefix-preserving mode that maintains network hierarchy

The functions implement the `ipcrypt-deterministic` and `ipcrypt-pfx`
algorithms from the IPCrypt specification.

Related PR: vectordotdev/vrl#1506
auto-merge was automatically disabled September 16, 2025 21:23

Head branch was pushed to by a user without write access

@pront pront enabled auto-merge September 17, 2025 00:10
auto-merge was automatically disabled September 17, 2025 10:02

Head branch was pushed to by a user without write access

@pront pront enabled auto-merge September 17, 2025 13:15
@alterstep
Copy link
Contributor Author

I'm really confused about the license file check.

@pront
Copy link
Member

pront commented Sep 17, 2025

I'm really confused about the license file check.

It is fixed now. I will work on this PR until it is merged. Thanks @alterstep

@pront pront added this pull request to the merge queue Sep 17, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 17, 2025
@pront pront added this pull request to the merge queue Sep 17, 2025
Merged via the queue into vectordotdev:master with commit 92781bd Sep 17, 2025
68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain: external docs Anything related to Vector's external, public documentation no-changelog Changes in this PR do not need user-facing explanations in the release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants