Skip to content

Add Tracing for PII file masking#812

Merged
yashwagle1 merged 1 commit intomainfrom
feat/TracesForFileMasking-v2
Apr 28, 2026
Merged

Add Tracing for PII file masking#812
yashwagle1 merged 1 commit intomainfrom
feat/TracesForFileMasking-v2

Conversation

@yashwagle1
Copy link
Copy Markdown
Contributor

No description provided.

@yashwagle1 yashwagle1 force-pushed the feat/TracesForFileMasking-v2 branch from 82a4cd4 to da40d6a Compare April 27, 2026 17:55
Comment thread src/uipath_langchain/agent/tools/internal_tools/pii_masker.py
# Re-upload redacted bytes to orchestrator so LLMOps traces can
# resolve a download URL for the masked attachment.
for idx, masked_file in enumerate(masked_files):
uploaded_id = await self._upload_masked_to_orchestrator(masked_file)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can this be parallelized?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I can do that in the next PR

Comment thread src/uipath_langchain/agent/tools/internal_tools/pii_masker.py Outdated
Copy link
Copy Markdown
Contributor

@saksharthakkar saksharthakkar left a comment

Choose a reason for hiding this comment

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

approving with comments...
you'll also need to bump up version and resolve CI errors

@yashwagle1 yashwagle1 force-pushed the feat/TracesForFileMasking-v2 branch 4 times, most recently from 40f007a to e954877 Compare April 28, 2026 06:44
@yashwagle1 yashwagle1 force-pushed the feat/TracesForFileMasking-v2 branch from e954877 to b95fbdb Compare April 28, 2026 06:48
@yashwagle1 yashwagle1 merged commit f7a9c84 into main Apr 28, 2026
42 of 45 checks passed
@yashwagle1 yashwagle1 deleted the feat/TracesForFileMasking-v2 branch April 28, 2026 06:55
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