Summary: | EDIFACT vendor account records sets default port incorrectly for FTP | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | Acquisitions | Assignee: | Kyle M Hall (khall) <kyle> |
Status: | Pushed to main --- | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
Severity: | major | ||
Priority: | P5 - low | CC: | david, martin.renvoize |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: |
This fixes the default upload and download ports when creating an EDI account (Koha administration > Acquisition parameters > EDI accounts > New account).
If:
- FTP is selected, it now defaults to port 21 (instead of port 22).
- SFTP is selected, it defaults to port 22.
|
|
Version(s) released in: |
25.05.00
|
Circulation function: | |
Attachments: |
Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP
Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP Bug 39878: (follow-up) Update hints for default FTP and SFTP ports Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP Bug 39878: (follow-up) Update hints for default FTP and SFTP ports |
Description
Kyle M Hall (khall)
2025-05-12 14:43:17 UTC
Created attachment 182297 [details] [review] Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP When creating or editing an EDI (EDIFACT vendor record) if the port is left empty it will be set to port 22. This is fine then the service is SFTP, but if FTP is selected the default to 21. Test plan: 1) Enable EDIFACT 2) Create a new EDI Vendor account with the FTP transport selected. Pleave the ports unpopulated 3) Note the saved record has port 22 set 4) Apply this patch 5) Restart all the things! 6) Repeat stepts 1 and 2 7) The port should now be 21 for new ftp site and 22 for new stp sites! Created attachment 182316 [details] [review] Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP When creating or editing an EDI (EDIFACT vendor record) if the port is left empty it will be set to port 22. This is fine then the service is SFTP, but if FTP is selected the default to 21. Test plan: 1) Enable EDIFACT 2) Create a new EDI Vendor account with the FTP transport selected. Pleave the ports unpopulated 3) Note the saved record has port 22 set 4) Apply this patch 5) Restart all the things! 6) Repeat stepts 1 and 2 7) The port should now be 21 for new ftp site and 22 for new stp sites! Signed-off-by: David Nind <david@davidnind.com> Created attachment 182317 [details] [review] Bug 39878: (follow-up) Update hints for default FTP and SFTP ports Hint now covers the default ports for FTP (21) and SFTP (22), instead of "will default to 22". Signed-off-by: David Nind <david@davidnind.com> Created attachment 182331 [details] [review] Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP When creating or editing an EDI (EDIFACT vendor record) if the port is left empty it will be set to port 22. This is fine then the service is SFTP, but if FTP is selected the default to 21. Test plan: 1) Enable EDIFACT 2) Create a new EDI Vendor account with the FTP transport selected. Pleave the ports unpopulated 3) Note the saved record has port 22 set 4) Apply this patch 5) Restart all the things! 6) Repeat stepts 1 and 2 7) The port should now be 21 for new ftp site and 22 for new stp sites! Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Created attachment 182332 [details] [review] Bug 39878: (follow-up) Update hints for default FTP and SFTP ports Hint now covers the default ports for FTP (21) and SFTP (22), instead of "will default to 22". Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Pushed for 25.05! Well done everyone, thank you! |