Skip to content

Disallow setting of readonly/internal parameters via mavftp #29988

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

peterbarker
Copy link
Contributor

@peterbarker peterbarker commented May 6, 2025

Closes #29985

This work sponsored by Freespace Solutions

@rmackay9 rmackay9 added the BUG label May 6, 2025
@peterbarker peterbarker force-pushed the pr/param-write-mavftp-reuse branch from b75038e to e15c638 Compare May 6, 2025 07:47
@peterbarker peterbarker force-pushed the pr/param-write-mavftp-reuse branch from e15c638 to 5bdfef5 Compare May 7, 2025 07:34
@peterbarker peterbarker force-pushed the pr/param-write-mavftp-reuse branch from 5bdfef5 to 122c3fa Compare May 7, 2025 08:19
Copy link
Contributor

@rmackay9 rmackay9 left a comment

Choose a reason for hiding this comment

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

sorry, I hate to be annoying but I'd just like to be sure the small requested comment fix goes in before merging

@peterbarker
Copy link
Contributor Author

sorry, I hate to be annoying but I'd just like to be sure the small requested comment fix goes in before merging

Comments re-added.

@peterbarker peterbarker force-pushed the pr/param-write-mavftp-reuse branch from 122c3fa to 322eb62 Compare May 9, 2025 05:12
@rmackay9 rmackay9 dismissed their stale review May 9, 2025 05:41

thanks very much for adding the comment

@peterbarker peterbarker force-pushed the pr/param-write-mavftp-reuse branch from 322eb62 to b572d75 Compare May 9, 2025 09:25
@peterbarker peterbarker force-pushed the pr/param-write-mavftp-reuse branch from b572d75 to b73ad13 Compare May 9, 2025 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

MAVFTP upload of parameters ignores ReadOnly flag
3 participants