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.

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

Testing #161

Open
wants to merge 2 commits into
base: master
from

Conversation

@Josue-T
Copy link
Collaborator

Josue-T commented Dec 1, 2019

Problem

  • Package is not up to date
  • Getting the option to choose a server_name at installation without further work from the user who install this package
    #92

Solution

  • Update the package
  • Add a .well-known delegation with a nginx redirection (location). I got help from tulir from the synapse room. If you would like to see picture of it working, I took ssome screenshot.
    Installation is ok and federation is working but I didn't try yet backup, upgrade and restore. I'll do that when the current testing PR will be merged into master.

PR Status

  • Code finished.
  • Tested with Package_check.
  • Fix or enhancement tested.
  • Upgrade from last version tested.
  • Can be reviewed and tested.

Validation


Minor decision

  • Upgrade previous version :
  • Code review :
  • Approval (LGTM) :
  • Approval (LGTM) :
  • CI succeeded :
    Build Status

When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.

Thatoo and others added 2 commits Dec 1, 2019
* Update README.md

* Update manifest.json

* a

* Delete a

* avoid boolean for public but ask a clear choice Yes/No

* Update check_process

Add server_name="domain2.tld"    (DOMAIN)

* .well-known

DNS conf become optionnal with .well-known edition

* .well-known redirection for access by federation

Create .well-known redirection for access by federation

* .well-known redirection for access by federation

If not existing, create .well-known redirection for access by federation

* Update README.md

back to 0.99.5.2

* back to 0.99.5.2

* back to 0.99.5.2

* back to 0.99.5.2

* Update install

* Update upgrade

* Update install

* Update upgrade

* Update install

* Update upgrade

* Update .well-known redirection

for access by federation and applications like Riot.im

* Update .well-known redirection

for access by federation and applications like Riot.im

* update to 1.1.0

* update to 1.1.0

* update to 1.1.0

* update to 1.1.0

* update to 1.1.0

* update to 1.1.0

* Update install

* Update upgrade

* Update to 1.2.1

* Update to 1.2.1

* Update to 1.2.1

* Back to v1.1.0

* Back to v1.1.0

* Back to v1.1.0

* Update to v1.2.1

* Update to v1.2.1

* Update to v1.2.1

* Important update of upgrade script

ynh_replace_string __SERVER_NAME__ $server_name "$homeserver_config_path" was missing

* Back to v1.1.0

* Back to v1.1.0

* Back to v1.1.0

* Fix typo for ipv6 validation

* Upgrade to v1.2.1

* Upgrade to v1.2.1

* Upgrade to v1.2.1

* remove server_name's nginx conf

remove /etc/nginx/conf.d/${server_name}.d/server_name.conf

* Create server_name.conf

To allow the automatic well-known system for server-name detection instead of DNS record

* Update install

In
# Create .well-known redirection for access by federation
change
  cat > /etc/nginx/conf.d/${server_name}.d/server_name.conf <<EOF
    location /.well-known/matrix/ {
        return 200 '{"m.server": "$domain", "m.homeserver": "https://$domain"}';
        add_header Content-Type application/json;
        add_header Access-Control-Allow-Origin '*';
        }
EOF
to 
cp ../conf/server_name.conf /etc/nginx/conf.d/${server_name}.d/server_name.conf
after creation of ../conf/server_name.conf file.

* Update upgrade

In
# Create .well-known redirection for access by federation if it doesn't exist
change
  cat > /etc/nginx/conf.d/${server_name}.d/server_name.conf <<EOF
    location /.well-known/matrix/ {
        return 200 '{"m.server": "$domain", "m.homeserver": "https://$domain"}';
        add_header Content-Type application/json;
        add_header Access-Control-Allow-Origin '*';
        }
EOF
to
cp ../conf/server_name.conf /etc/nginx/conf.d/${server_name}.d/server_name.conf

* backup well.known server_name nginx cond

add
ynh_backup "/etc/nginx/conf.d/${server_name}.d/server_name.conf"

* Backup well.known server-name nging conf

add
server_name=$(ynh_app_setting_get $app server_name) 
to make
ynh_backup "/etc/nginx/conf.d/${server_name}.d/server_name.conf"
understood

* Update backup

* Update scripts/install

Co-Authored-By: Josue-T <josue@tille.ch>

* Update install

* Update README.md

Co-Authored-By: Josue-T <josue@tille.ch>

* Update scripts/upgrade

Co-Authored-By: Josue-T <josue@tille.ch>

* Update conf/server_name.conf

Co-Authored-By: Josue-T <josue@tille.ch>

* move .well-known redirection to config section

move #Create .well-known redirection for access by federation to end of config section

* move .well-known redirection to config section 

move #Create .well-known redirection to the end of config section and Indentation

* add checksum management for server_name.conf

add checksum management for server_name.conf with the helper ynh_store_file_checksum

* add checksum management for server_name.conf

add checksum management for server_name.conf with the helper ynh_store_file_checksum

* replace __SERVER_NAME__

* replace __SERVER_NAME__

* delete a blank line

* Update upgrade

* Update install

* Update check_process

Co-Authored-By: Josue-T <josue@tille.ch>

* Update conf/server_name.conf

Co-Authored-By: Josue-T <josue@tille.ch>

* remove blank line

* ynh_store_file_checksum after cp inside if block

* change order checksum and cp

change order to ynh_backup_if_checksum_is_different then cp then ynh_store_file_checksum

* Update install

* Update upgrade

* Update backup

* Update backup

* Update scripts/backup

Co-Authored-By: Josue-T <josue@tille.ch>

* Update scripts/install

Co-Authored-By: Josue-T <josue@tille.ch>

* Update scripts/install

Co-Authored-By: Josue-T <josue@tille.ch>

* Update scripts/install

Co-Authored-By: Josue-T <josue@tille.ch>

* Update install

* Update remove

* Update upgrade
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.