Hi all, Bumping this thread, since it has had no replies on the list yet. Short recap of where SIP-230 stands:
- SIP: https://www.google.com/url?q=https://github.com/apache/superset/issues/44512&source=gmail&ust=1792604297037000&sa=E - Reference PR: https://www.google.com/url?q=https://github.com/apache/superset/pull/44499&source=gmail&ust=1792604297037000&sa=E (just brought up to date with master; it is parked under hold:sip! until this is settled) Feedback so far has come in on the GitHub issue: one +1, plus a question on whether this should be a config key rather than a feature flag. I agree it is closer to a long-lived deployment policy than a rollout toggle, so I'm happy to switch to a config key if that is the preferred direction. I used a feature flag only because flags already reach the frontend without extra plumbing. Questions I'd especially like input on: 1. Feature flag vs. config key for REQUIRE_DOWNLOAD_REASON. 2. Recording the reason in the existing Action Log entry (no new table) - any concerns? 3. Anything missing from the scope (Explore, dashboard charts, dashboard-wide Excel, SQL Lab, scheduled reports)? If there are no objections by Oct 28, I'll open the [VOTE] thread. Thanks, Seunggabi (seunggabi) On Fri, Oct 02, 2026 11:53 AM, 김승갑 <[email protected]> wrote: > Hi all, > > I'd like to open discussion on SIP-230: an opt-in download reason for > CSV/XLSX exports. Deployments that hold personal data often need to > record why a file was exported, not just who/what/when — Korea's > PIPA safeguards standard and GDPR/HIPAA-style audits ask the same > "purpose of processing" question. > > Behind a new feature flag (REQUIRE_DOWNLOAD_REASON, default False), > the UI prompts for a short reason before CSV/XLSX export (Explore, > dashboard charts, dashboard-wide Excel, SQL Lab), the backend rejects > export requests without one, and the reason rides along in the > existing Action Log entry — no new table. Scheduled reports supply > their own reason automatically. Flag off, nothing changes. > > SIP: https://www.google.com/url?q=https://github.com/apache/ > superset/issues/44512&source=gmail&ust=1790996007510000&sa=E > Reference PR: https://www.google.com/url?q=https://github.com/apache/ > superset/pull/44499&source=gmail&ust=1790996007510000&sa=E > > Would love to hear your thoughts. > > Thanks, > Seunggabi (seunggabi) >
