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

1.7: postgresql crash #1479

Closed
chint95 opened this issue May 2, 2020 · 4 comments
Closed

1.7: postgresql crash #1479

chint95 opened this issue May 2, 2020 · 4 comments

Comments

@chint95
Copy link

chint95 commented May 2, 2020

After #1469 the postgresql container crashes on my ipv4 only system with:

CONTEXT: line 85 of configuration file "/data/pg_hba.conf"
FATAL: could not load pg_hba.conf

Setting SUBNET6 env seems to resolve the problem.
@muhlemmer
Copy link
Member

My sincere apologies. For now, I will revert the merge of #1469. We need to implement a conditional for this variable.

This was referenced May 3, 2020
Merged
Merged
bors bot added a commit that referenced this issue May 5, 2020
@bors @muhlemmer
Merge #1481
535562e 
1481: Postgresql: default SUBNET6 in pg_hba r=mergify[bot] a=muhlemmer

## What type of PR?

Bug fix

## What does this PR do?

When PostgreSQL for Mailu is used in IPv4 only, without IPv6, `SUBNET6` is unset and causes an empty field in the `pg_hba.conf` file.  PostgreSQL refuses to start in this case.

### Related issue(s)
- Bug was introduced in #1257, merged recently
- 1.7 user reported PostgreSQL connection errors on IPv6 in #1479
- #1257 was Backported to 1.7 in #1469 
- Users on 1.7 reported this broke IPv4 installation in #1479 
- Backport was reverted in #1480 

Sorry for the mess 😒 

**Note: ** if this patch seems to be stable, I will re-initiate the backport.

## Prerequistes
Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

- [x] In case of feature or enhancement: documentation updated accordingly
- [x] Unless it's docs or a minor change: add [changelog](https://mailu.io/master/contributors/guide.html#changelog) entry file.


Co-authored-by: Tim Möhlmann <muhlemmer@gmail.com>
@Nebukadneza
Copy link
Member

Hi There,

The Mailu-Project is currently in a bit of a bind! We are short on man-power, and we need to judge if it is possible for us to put in some work on this issue.

To help with that, we are currently trying to find out which issues are actively keeping users from using Mailu, which issues have someone who want to work on them — and which issues may be less important. These a less important ones could be discarded for the time being, until the project is in a more stable and regular state once again.

In order for us to better assess this, it would be helpful if you could put a reaction on this post (use the 😃 icon to the top-right).

  • 👍️ if you need this to be able to use Mailu. Ideally, you’d also be able to test this on your installation, and provide feedback …
  • 🎉 if you find it a nice bonus, but no deal-breaker
  • 🚀 if you want to work on it yourself!
    We want to keep this voting open for 2 weeks from now, so please help out!

@stale
Copy link

stale bot commented Sep 23, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status/response_needed Waiting for a response from the author label Sep 23, 2020
@Diman0 Diman0 removed the status/response_needed Waiting for a response from the author label Sep 25, 2020
@Diman0
Copy link
Member

Diman0 commented Sep 25, 2020

Hi There,

we see this issue had only little attention 😟. As much as it pains us: In order to get the current issues a bit more manageable for us, we decided to close it. 👋👋

We hope that no one feels offended by doing so. Should the issue get really pressing in the future, please feel free to re-open it.

Thank you for your patience and understanding, 🙇

  • Your Mailu Team

@Diman0 Diman0 closed this as completed Sep 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@muhlemmer @Nebukadneza @chint95 @Diman0