Skip to content

Added support to connect to Mapepire Server#3236

Merged
buzzia2001 merged 6 commits into
masterfrom
mapepireServer
Jun 4, 2026
Merged

Added support to connect to Mapepire Server#3236
buzzia2001 merged 6 commits into
masterfrom
mapepireServer

Conversation

@sebjulliand
Copy link
Copy Markdown
Member

@sebjulliand sebjulliand commented Jun 1, 2026 •
edited
Loading

Changes

This PR adds support for connecting to a Mapepire Server running on the remote IBM i instead of running it in single mode through SSH.

Three new settings have been added to the connection settings:
image

  • Connect to remote Mapepire server: switch to connecting to Mapepire over HTTPS
  • Allow all certificates: allows to connect using a self-signed certificate
  • Remote Mapepire Server port: set the port to connect to. Defaults to 8076.

Enabling Connect to remote Mapepire server will connect Code for i to Mapepire using an HTTPS connection and skip uploading Mapepire's jar file.

How to test this PR

  1. Start Mapepire as a server on IBM i
  2. Change the connection settings to enable Connect to Mapepire over HTTPS (and Allow all certificates)
  3. Connect
  4. The connection must succeed (and be faster than usual)

Checklist

  • have tested my change

@sebjulliand
Added support to connect to Mapepire Server
d20ffc2 
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
@sebjulliand sebjulliand requested a review from a team June 1, 2026 14:10
@sebjulliand sebjulliand self-assigned this Jun 1, 2026
@sebjulliand sebjulliand added the enhancement New feature or request label Jun 1, 2026
@sebjulliand sebjulliand temporarily deployed to testing_environment June 1, 2026 14:10 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026 •
edited
Loading

👋 A new build is available for this PR based on b2a4bf8.

@sebjulliand
Changed labels to use HTTPS
a8593ec 
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
@sebjulliand sebjulliand temporarily deployed to testing_environment June 1, 2026 14:16 — with GitHub Actions Inactive
@buzzia2001
Copy link
Copy Markdown
Member

buzzia2001 commented Jun 1, 2026

Hi @sebjulliand ,
I’ll take a look in the next few days... but I’d like to discuss with you the feasibility of taking it a step further... Let me explain: why not consider using an existing Mapepire server if one is already running on the system, rather than setting up new ones? This would significantly reduce the initial setup time if Mapepire is already active. Consider, for example, that we’re currently using it on systems with development

@sebjulliand
Copy link
Copy Markdown
Member Author

sebjulliand commented Jun 1, 2026

@buzzia2001 I guess we're in agreement then; this is exactly the point of this PR: to connect to a Mapepire Server already running on the partition 😅
It won't start a new process, it will connect to an existing one.

buzzia2001
buzzia2001 requested changes Jun 2, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@buzzia2001 buzzia2001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's great, even though saying “HTTPS” might be misleading, at least it was for me. I don't know...
Let's just say that if you already have the service running, you avoid having to upload files to individual home directories, and in terms of performance, you skip the JVM startup time, so that's a big deal.

Comment thread src/extension.ts Outdated
@sebjulliand @buzzia2001
Fixed placeholders
268be38 
Co-authored-by: Andrea Buzzi <155985472+buzzia2001@users.noreply.github.com>
@sebjulliand sebjulliand temporarily deployed to testing_environment June 4, 2026 18:29 — with GitHub Actions Inactive
@sebjulliand
Updated settings label for Mapepire server connection
1f0cbb6 
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
@sebjulliand
Added support for JDBC options in newJob method
504b230 
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
@sebjulliand
Merge branch 'mapepireServer' of github.com:codefori/vscode-ibmi into…
b2a4bf8 
… mapepireServer
@sebjulliand sebjulliand temporarily deployed to testing_environment June 4, 2026 18:39 — with GitHub Actions Inactive
@sebjulliand sebjulliand requested a review from buzzia2001 June 4, 2026 18:39
@sebjulliand
Copy link
Copy Markdown
Member Author

sebjulliand commented Jun 4, 2026 •
edited
Loading

@buzzia2001 How about now?
image

@sebjulliand sebjulliand mentioned this pull request Jun 4, 2026
Open
buzzia2001
buzzia2001 approved these changes Jun 4, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@buzzia2001 buzzia2001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sebjulliand nice job, faster and faster

Image

@buzzia2001 buzzia2001 merged commit f1c91aa into master Jun 4, 2026
5 checks passed
@sebjulliand sebjulliand deleted the mapepireServer branch June 4, 2026 20:39
@sebjulliand
Copy link
Copy Markdown
Member Author

sebjulliand commented Jun 4, 2026

Much obliged @buzzia2001 🙏🏻

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

Reviewers

@buzzia2001 buzzia2001 buzzia2001 approved these changes

Assignees

@sebjulliand sebjulliand

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sebjulliand @buzzia2001