Skip to content

Improve http server code#14489

Merged
Siedlerchr merged 6 commits into
mainfrom
fix-http-server
Dec 1, 2025
Merged

Improve http server code#14489
Siedlerchr merged 6 commits into
mainfrom
fix-http-server

Conversation

@koppor

@koppor koppor commented Dec 1, 2025

Copy link
Copy Markdown
Member

Steps to test

Run http server

Maybe use some .http files from jabsrv/src/test

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • [/] I manually tested my changes in running JabRef (always required)
  • [/] I added JUnit tests for changes (if applicable)
  • [/] I added screenshots in the PR description (if change is visible to the user)
  • [/] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • [/] I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

@koppor koppor requested a review from palukku December 1, 2025 18:25
@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Dec 1, 2025
@Siedlerchr Siedlerchr added this pull request to the merge queue Dec 1, 2025
Merged via the queue into main with commit 636b37c Dec 1, 2025
58 of 64 checks passed
@Siedlerchr Siedlerchr deleted the fix-http-server branch December 1, 2025 20:39
koppor added a commit that referenced this pull request Dec 1, 2025
shubhamk0205 pushed a commit to shubhamk0205/jabref that referenced this pull request Dec 3, 2025
* Fix logging placeholder mismatch

* Remove wrong empty line

* Fix wrong space

* Route prefernces through

* Use OneConstant instead of Factor for preferences

* Remove obsolete file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants