PeachCloud/peach-workspace
3
2
Fork 1
Code Issues 17 Pull Requests 1 Projects Releases Wiki Activity

Insert domain into generated invites #124

New Issue
Closed
opened 2022-06-27 20:42:15 +00:00 by notplants · 3 comments
notplants commented 2022-06-27 20:42:15 +00:00
Owner
Copy Link
Copy Source

Invites currently take this form:
[::]:8009:@fxKem3e9v0VyU2+Nv/DiGC5RvdLA9HfmjJmT02Zw75k=.ed25519~R6F/TT6Xo6HdUcsdR+xXm8hH20gGkReBtOMNpeLyCYs=

We should look for the configured domain, and find and replace [::] with the configured domain, if it is configured.

Then in a yunohost install, we should also be able to configure the domain through environental variables, so yunohost can provide a domain to peachpub during install.

Invites currently take this form: [::]:8009:@fxKem3e9v0VyU2+Nv/DiGC5RvdLA9HfmjJmT02Zw75k=.ed25519~R6F/TT6Xo6HdUcsdR+xXm8hH20gGkReBtOMNpeLyCYs= We should look for the configured domain, and find and replace [::] with the configured domain, if it is configured. Then in a yunohost install, we should also be able to configure the domain through environental variables, so yunohost can provide a domain to peachpub during install.
👍 1
glyph commented 2022-06-28 10:31:51 +00:00
Owner
Copy Link
Copy Source

@notplants

Nice, this would be a great addition! Would it be as simple as calling the domainname Linux command, parsing the return value and replacing [::] with the value if it's not (none)?

@notplants Nice, this would be a great addition! Would it be as simple as calling the `domainname` Linux command, parsing the return value and replacing `[::]` with the value if it's not `(none)`?
notplants commented 2022-06-29 20:58:45 +00:00
Author
Owner
Copy Link
Copy Source

when I run domainname on my yunohost instance it returns (none). Additionally, on yunohost, you can configure many different apps with different domains, so I don't think domainname could work in that context.

In this PR (#125) I used a configured variable that I created while working on dynamic DNS. It appears to be working, and is easy for me to configure as required during the yunohost install script.

when I run domainname on my yunohost instance it returns (none). Additionally, on yunohost, you can configure many different apps with different domains, so I don't think domainname could work in that context. In this PR (https://git.coopcloud.tech/PeachCloud/peach-workspace/pulls/125) I used a configured variable that I created while working on dynamic DNS. It appears to be working, and is easy for me to configure as required during the yunohost install script.
👍 1
notplants commented 2022-06-30 13:01:58 +00:00
Author
Owner
Copy Link
Copy Source

closed via this issue #125

closed via this issue https://git.coopcloud.tech/PeachCloud/peach-workspace/pulls/125
🎉 1
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
 documentation
duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 maintenance
Pruning and weeding
 peach-lib
peach-network
peach-oled
peach-stats
peach-web
question
More information is needed
 refactor
Code rewrite
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
cblgh glyph (glyph) notplants
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: PeachCloud/peach-workspace#124