jenkinsci / jenkinsci/github-branch-source-plugin

[JENKINS-76053] Cannot delete configured GHE servers

Open
#1,473 0 comments 0 reactions 0 assignees View on GitHub
component:github-branch-source-plugin imported-jira-issue priority:major resolution:unresolved
Dominant language
Java
Stars
217
Forks
398
Avg merge
30m
Merged PRs (30d)
1

Description

Steps to reproduce



  1. Go to http://localhost:8080/manage/configure

  2. Below " GitHub Enterprise Servers ", click "Add"

  3. Because JENKINS-76052 confused you, enter "https://api.github.com" as URL and "Actual GitHub" as the name.

  4. Click Save.

  5. Reload the page after realizing that's not how you make GitHub available in the text field of JENKINS-76052.

  6. Remove "Actual GitHub", leaving the list empty.

  7. Save

  8. Reload the page.

Expected results

No servers

Actual results

Still there

Notes

Looks like https://reports.jenkins.io/core-taglib/jelly-taglib-ref.html#form.3ArepeatableProperty applies here.


Due to a bug in Stapler data binding the model elements are only set if they consist of one or more values. If all values have been removed in the user interface (i.e. the associated form is empty), then the setter is not invoked anymore. As a workaround, you need to override the corresponding configure method and clear the model property manually before invoking the data binding.

---
Originally reported by danielbeck, imported from: Cannot delete configured GHE servers


  • status: Open
  • priority: Major
  • component(s): github-branch-source-plugin
  • resolution: Unresolved
  • votes: 0
  • watchers: 1
  • imported: 2025-12-02

Raw content of original issue

Steps to reproduce



  1. Go to http://localhost:8080/manage/configure

  2. Below " GitHub Enterprise Servers ", click "Add"

  3. Because JENKINS-76052 confused you, enter "https://api.github.com" as URL and "Actual GitHub" as the name.

  4. Click Save.

  5. Reload the page after realizing that's not how you make GitHub available in the text field of JENKINS-76052.

  6. Remove "Actual GitHub", leaving the list empty.

  7. Save

  8. Reload the page.

Expected results

No servers

Actual results

Still there

Notes

Looks like https://reports.jenkins.io/core-taglib/jelly-taglib-ref.html#form.3ArepeatableProperty applies here.


Due to a bug in Stapler data binding the model elements are only set if they consist of one or more values. If all values have been removed in the user interface (i.e. the associated form is empty), then the setter is not invoked anymore. As a workaround, you need to override the corresponding configure method and clear the model property manually before invoking the data binding.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.