Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update stalwart.json - add port definition for web admin interface #4556

Merged
merged 10 commits into from
May 24, 2024
Merged

Update stalwart.json - add port definition for web admin interface #4556

merged 10 commits into from
May 24, 2024

Conversation

kri164
Copy link
Contributor

@kri164 kri164 commented Apr 17, 2024

Add port definition for web admin interface

@kri164
Update readme.md - info about new web admin interface
6bbfca6 
Add info about new web admin interface.

Signed-off-by: kri164 <52274164+kri164@users.noreply.github.com>
@kri164
Update stalwart.json - add port definition for web admin interface
e2055b6 
Add port definition for web admin interface

Signed-off-by: kri164 <52274164+kri164@users.noreply.github.com>
@docjyJ docjyJ mentioned this pull request Apr 23, 2024
Merged
@docjyJ
Copy link
Collaborator

docjyJ commented Apr 23, 2024

Yes !!!
Maybe update internal port "internal_port": "8080", ?

@docjyJ
Copy link
Collaborator

docjyJ commented Apr 23, 2024

Could we use the official image?
It is possible to add certificates from the admin interface.

If you already have a TLS certificate for your server, you can upload it in the Settings > Server > TLS > Certificates section.
See : https://stalw.art/docs/install/docker/

@szaimen szaimen mentioned this pull request Apr 24, 2024
Closed
21 tasks
@docjyJ docjyJ added 2. developing Work in progress enhancement New feature or request labels Apr 24, 2024
This was referenced Apr 27, 2024
Closed
Merged
@docjyJ docjyJ mentioned this pull request May 9, 2024
Merged
@docjyJ
Update stalwart.json
140167a 
See :
- marcoambrosini/aio-stalwart#15
- szaimen/aio-caddy#34

Signed-off-by: Jean-Yves <7360784+docjyJ@users.noreply.github.com>
@docjyJ
Merge branch 'refs/heads/patch-2' into patch-3
1b5b839
@docjyJ
Add warning
07d55b9 
Signed-off-by: Jean-Yves <7360784+docjyJ@users.noreply.github.com>
@docjyJ docjyJ marked this pull request as draft May 12, 2024 14:12
@docjyJ
Fix warn
581e69a 
Signed-off-by: Jean-Yves <7360784+docjyJ@users.noreply.github.com>
@docjyJ
Copy link
Collaborator

docjyJ commented May 12, 2024

I added a header to the configuration file:

Warning

Nextcloud Community Containers are beta features.
The Stalwart server is under development.

The stability of these two services is not guaranteed.
Do not use this feature as a main mail server without a redundancy system and without knowledge.

To learn or use as a secondary server enjoy it and please report bugs at marcoambrosini/aio-stalwart.

Link to related Pull Requests:

Stalwart is moving to version 0.8, I think we should wait for this update before merging.
And we can write a tutorial to configure Nextcloud Mail, a bit like for LLDAP.

It would be good to put some links to tutorials for configuring a DNS zone and adding MX redundancy.

What do you think?

@szaimen
Copy link
Collaborator

szaimen commented May 13, 2024

I added a header to the configuration file:

Warning

Nextcloud Community Containers are beta features. The Stalwart server is under development.

The stability of these two services is not guaranteed. Do not use this feature as a main mail server without a redundancy system and without knowledge.

To learn or use as a secondary server enjoy it and please report bugs at marcoambrosini/aio-stalwart.

Link to related Pull Requests:

Stalwart is moving to version 0.8, I think we should wait for this update before merging. And we can write a tutorial to configure Nextcloud Mail, a bit like for LLDAP.

It would be good to put some links to tutorials for configuring a DNS zone and adding MX redundancy.

What do you think?

Sounds good! Thanks for handling this :)

@docjyJ
add upgrading.md
2117809 
Signed-off-by: Jean-Yves <7360784+docjyJ@users.noreply.github.com>
@docjyJ
Copy link
Collaborator

docjyJ commented May 19, 2024 •
edited

The new version 0.8 has just been published.
It includes breaking of data structure change.

I tried to update my own server, but it was failed. I am stupid, I did not save 😞. It doesn't matter, I have nothing important.

@szaimen
Copy link
Collaborator

szaimen commented May 24, 2024

Hey, I am wondering shouldn't we merge the change here already so that people can use v0.7.0? Then you could do the update to v0.8.0 in a follow-up...

@docjyJ
Copy link
Collaborator

docjyJ commented May 24, 2024

Yes, it seems good to me.
I don't have enough time, but I managed to migrate my server!

To test in beta.

@docjyJ docjyJ marked this pull request as ready for review May 24, 2024 14:04
@szaimen szaimen requested a review from docjyJ May 24, 2024 14:12
@szaimen szaimen mentioned this pull request May 24, 2024
Closed
44 tasks
@docjyJ docjyJ requested a review from szaimen May 24, 2024 14:14
docjyJ
docjyJ approved these changes May 24, 2024
View reviewed changes
community-containers/stalwart/readme.md Outdated Show resolved Hide resolved
community-containers/stalwart/upgrading.md Outdated Show resolved Hide resolved
@docjyJ @szaimen
Update community-containers/stalwart/readme.md
e888a76 
Co-authored-by: Simon L. <szaimen@e.mail.de>
Signed-off-by: Jean-Yves <7360784+docjyJ@users.noreply.github.com>
@docjyJ
Update upgrading.md
6e083ce 
Signed-off-by: Jean-Yves <7360784+docjyJ@users.noreply.github.com>
@docjyJ
Copy link
Collaborator

docjyJ commented May 24, 2024

  • TODO: test 🙃

@szaimen
fix typo
845be45 
Signed-off-by: Simon L <szaimen@e.mail.de>

Signed-off-by: Simon L. <szaimen@e.mail.de>
@szaimen szaimen merged commit 9e80bcf into nextcloud:main May 24, 2024
4 checks passed
@szaimen
Copy link
Collaborator

szaimen commented May 24, 2024

Thanks a lot! :)

@szaimen szaimen added this to the next milestone May 24, 2024
@szaimen szaimen added 3. to review Waiting for reviews dependencies and removed 2. developing Work in progress labels May 24, 2024
@szaimen
Copy link
Collaborator

szaimen commented May 30, 2024

This is now released with v8.3.0 Beta. Testing and feedback is welcome! See https://github.com/nextcloud/all-in-one#how-to-switch-the-channel

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

@docjyJ docjyJ docjyJ approved these changes

@szaimen szaimen szaimen left review comments

@hgterra hgterra hgterra left review comments

Assignees
No one assigned
Labels
3. to review Waiting for reviews dependencies enhancement New feature or request
Projects
None yet
Milestone
v8.3.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@kri164 @docjyJ @szaimen @hgterra