-
Notifications
You must be signed in to change notification settings - Fork 0
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
reports: finalize database connection isolation #123
Comments
See #96 for background, this filter is necessary to protect the system. If you're happy with a report, you can just let me know how to find it and I can upgrade. Nothing will change, I'll just check that the SQL isn't doing anything crazy and move it to a protected place. Anyone is of course welcome to the new role as well, it basically boils down to agreeing to develop in test and not bringing anything to production until it's been tested. https://handbook.arctosdb.org/documentation/reports.html will continue to develop as we work through this. |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
As has been stated in other issues and meetings, a word of reassurance-- there will be no changes to your reports-- we are moving SQL to more secure fields and in this transition you may see red boxes, skulls, green checks, etc as these measures are completed. We dont want your reports to change either! There's no action on your part needed and you can continue to print as needed. But since Dusty is also doing QA/QC on SQL this is a multistep process. As usual, please file an issue if you want a new report! |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Tech chat with @mkoo - will contact heavy users, check back re: blocking unfiltered execution 20250201. To get rid of the scary icon in the reporter, just let me know how to find your needed reports in whatever way works best for you. |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
@dustymc Are we able to edit the tags once they've been changed to the new SQL, or should we ensure the tags are finished before requesting help? thanks! |
@Kmullineaux https://arctos.database.museum/Admin/user_roles.cfm#write_report can alter anything but SQL, https://arctos.database.museum/Admin/user_roles.cfm#write_sql can alter SQL. The recommendation will always be to start by gathering the data then working on layout no matter what the current environment look like or who's involved. I'm happy to help with SQL anywhere along the way and can optimize around requested data rather than current usage, just let me know how I can help. |
@dustymc I've created a new issue for NHSM reports so I'm not filling up this one! |
Need Help?
If you require assistance rewriting reports, just ask.
Source
#96
Summary
#96 introduces isolated SQL and detects inappropriate database connections, but does not prevent executing reports with them. The new structure cannot prevent ArctosDB/arctos#8342 until this is completed, but the desire is to be as non-disruptive as possible so a stage two is necessary.
User Interface Changes
The red warning of next release will become a hard stop when this is implemented.
Data Structure Changes
none
The text was updated successfully, but these errors were encountered: