Skip to content

fix: debug drf rpcapi#9096

Merged
jennifer-richards merged 5 commits intoietf-tools:feat/rpc-api-drffrom
jennifer-richards:drf-fixes-for-xfer
Jul 5, 2025
Merged

fix: debug drf rpcapi#9096
jennifer-richards merged 5 commits intoietf-tools:feat/rpc-api-drffrom
jennifer-richards:drf-fixes-for-xfer

Conversation

@jennifer-richards
Copy link
Copy Markdown
Member

@jennifer-richards jennifer-richards commented Jul 4, 2025

Fixes a few bugs that were discovered while reconciling the code with the rfced-xfer project.

The refactor in cd01a16 renders unnecessary the earlier change to the DocumentInfo.title validation, so this PR reverts that change. The code on this branch has not been deployed anywhere, so we can just remove the migration rather than reversing it in a new one.

Comment thread ietf/utils/validators.py
regex="^[^\x01-\x1f]*$",
message="Please enter a string without control characters.",
)
regex="^[^\x00-\x1f]*$",
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.

indentation looks off

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes indeed, good catch. However, I'm inclined to leave it because this is reverting back to the strangely indented original on the main branch. When we squash-merge the feat/rpc-api-drf branch back to main, the net result will leave the lines alone so I think it's better not to link it to this PR in the history.

@jennifer-richards jennifer-richards merged commit e6f76a4 into ietf-tools:feat/rpc-api-drf Jul 5, 2025
2 checks passed
@jennifer-richards jennifer-richards deleted the drf-fixes-for-xfer branch July 5, 2025 02:49
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants