Skip to content

Add request header configuration to admin dataset source interface#127

Merged
rajkowski merged 7 commits intomainfrom
copilot/extend-datasets-database
Feb 15, 2026
Merged

Add request header configuration to admin dataset source interface#127
rajkowski merged 7 commits intomainfrom
copilot/extend-datasets-database

Conversation

Copy link
Contributor

Copilot AI commented Feb 14, 2026

Request header configuration was only available in the Visual Data Editor. Admin users working in /admin/dataset-source lacked access to configure authentication headers for dataset downloads.

Changes

JSP View (dataset-source.jsp)

  • Added Request Headers fieldset with dynamic header management
  • JavaScript handles initialization from existing JSON, add/remove rows, and serialization on submit
  • Foundation grid layout (5-6-1) matches existing admin forms

Widget Controller (DatasetSourceWidget.java)

  • Added dataset.setRequestConfig(context.getParameter("requestConfig")) to updateSettings() and downloadRemoteFileAction() methods
  • Headers persist with both "Save" and "Save & Download Remote File" actions

Implementation

Headers stored as JSONB and automatically used by existing DatasetDownloadRemoteFileCommand:

{
  "headers": {
    "Authorization": "******",
    "X-API-Key": "abc123xyz"
  }
}

UI provides empty state, dynamic add/remove, and HTML escaping. Backend integration leverages existing infrastructure - no changes needed to download logic.

Original prompt

In the Visual Data Editor, users can choose a URL to download data from and synchronize the datasets with... but extra inputs might be needed for Request Headers like an access token or other values. Extend the datasets database with a JSONB field for storing request information, and make the dynamic form for entering the values. Information could include Request Method and OUATH sequence.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 14, 2026 19:20
…eaders

Co-authored-by: rajkowski <10373492+rajkowski@users.noreply.github.com>
Co-authored-by: rajkowski <10373492+rajkowski@users.noreply.github.com>
Copilot AI changed the title [WIP] Extend datasets database with JSONB field for request info Add request configuration support for dataset HTTP synchronization Feb 14, 2026
Copilot AI requested a review from rajkowski February 14, 2026 19:29
Co-authored-by: rajkowski <10373492+rajkowski@users.noreply.github.com>
Copilot AI changed the title Add request configuration support for dataset HTTP synchronization Add request header configuration to admin dataset source interface Feb 15, 2026
@rajkowski rajkowski marked this pull request as ready for review February 15, 2026 21:45
@rajkowski rajkowski merged commit 53c1873 into main Feb 15, 2026
5 checks passed
@rajkowski rajkowski deleted the copilot/extend-datasets-database branch February 15, 2026 21:46
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