Bugzilla – Attachment 182316 Details for
Bug 39878
EDIFACT vendor account records sets default port incorrectly for FTP
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP
Bug-39878-EDIFACT-vendor-account-records-sets-defa.patch (text/plain), 2.29 KB, created by
David Nind
on 2025-05-12 22:02:45 UTC
(
hide
)
Description:
Bug 39878: EDIFACT vendor account records sets default port incorrectly for FTP
Filename:
MIME Type:
Creator:
David Nind
Created:
2025-05-12 22:02:45 UTC
Size:
2.29 KB
patch
obsolete
>From b0cbe0078a2a378213aeb40e835009f1c885b3f7 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Mon, 12 May 2025 14:48:17 +0000 >Subject: [PATCH] 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> >--- > admin/edi_accounts.pl | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > >diff --git a/admin/edi_accounts.pl b/admin/edi_accounts.pl >index 4e58de294b..410bcb8eaf 100755 >--- a/admin/edi_accounts.pl >+++ b/admin/edi_accounts.pl >@@ -75,12 +75,12 @@ if ( $op eq 'acct_form' ) { > my $password = scalar $input->param('password'); > $password = $crypt->encrypt_hex($password); > my $fields = { >- description => scalar $input->param('description'), >- host => scalar $input->param('host'), >- username => scalar $input->param('username'), >- password => $password, >- upload_port => scalar $input->param('upload_port') || 22, >- download_port => scalar $input->param('download_port') || 22, >+ description => scalar $input->param('description'), >+ host => scalar $input->param('host'), >+ username => scalar $input->param('username'), >+ password => $password, >+ upload_port => scalar $input->param('upload_port') || $input->param('transport') eq 'FTP' ? 21 : 22, >+ download_port => scalar $input->param('download_port') || $input->param('transport') eq 'FTP' ? 21 : 22, > vendor_id => scalar $input->param('vendor_id'), > upload_directory => scalar $input->param('upload_directory'), > download_directory => scalar $input->param('download_directory'), >-- >2.39.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 39878
:
182297
|
182316
|
182317
|
182331
|
182332