You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
dd12384f22
* Fix --port setting Unfortunately there was an error in #13195 which set the --port option before the settings were read. This PR fixes this by moving applying this option to after the the settings are read However, on looking further into this code I believe that the setPort code was slightly odd. Firstly, it may make sense to run the install page on a different temporary port to the full system and this should be possible with a --install-port option. Secondy, if the --port option is provided we should apply it to both otherwise there will be unusual behaviour on graceful restart Thirdly, the documentation for --port says that the setting is temporary - it should therefore not save its result to the configuration (This however, does mean that authorized_keys and internal links may not be correct. - I think we need to discuss this option further.) Fix #13277 Signed-off-by: Andrew Thornton <art27@cantab.net> * Update cmd/web.go * Apply suggestions from code review Co-authored-by: techknowlogick <techknowlogick@gitea.io> |
4 years ago | |
|---|---|---|
| .. | ||
| admin.go | 4 years ago | |
| admin_auth_ldap.go | 4 years ago | |
| admin_auth_ldap_test.go | 4 years ago | |
| cert.go | 5 years ago | |
| cmd.go | 4 years ago | |
| convert.go | 5 years ago | |
| doctor.go | 4 years ago | |
| dump.go | 4 years ago | |
| embedded.go | 4 years ago | |
| embedded_stub.go | 4 years ago | |
| generate.go | 4 years ago | |
| hook.go | 4 years ago | |
| keys.go | 5 years ago | |
| mailer.go | 4 years ago | |
| manager.go | 4 years ago | |
| migrate.go | 5 years ago | |
| migrate_storage.go | 4 years ago | |
| serv.go | 4 years ago | |
| web.go | 4 years ago | |
| web_graceful.go | 4 years ago | |