Skip to content

enh: add exapps info endpoint for HaRP#505

Merged
oleksandr-nc merged 53 commits intomainfrom
enh/harp/exapp-info-endpoint
Apr 1, 2025
Merged

enh: add exapps info endpoint for HaRP#505
oleksandr-nc merged 53 commits intomainfrom
enh/harp/exapp-info-endpoint

Conversation

@kyteinsky
Copy link
Collaborator

@kyteinsky kyteinsky commented Feb 7, 2025 •
edited by oleksandr-nc
Loading

@kyteinsky kyteinsky requested a review from bigcat88 February 7, 2025 00:03
bigcat88
bigcat88 reviewed Feb 7, 2025
View reviewed changes
@kyteinsky kyteinsky force-pushed the enh/harp/exapp-info-endpoint branch 2 times, most recently from ff850b2 to c6067d2 Compare February 14, 2025 10:43
@kyteinsky
enh: add exapps info endpoint for HaRP
ebe4a00 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
get metadata for only one ex-app
6604b9e 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
add bruteforce throttler
4b38bc2 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
enh: add user info endpoint for HaRP
e409bdb 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
enh: add user info endpoint for HaRP
c3c2cd5 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
fixes
0ae2f24 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
adjustments
8b8dc22 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
occ: make daemon commands harp combatible
5884ab7 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
connect to internal dsp of harp if present
f21b930 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky kyteinsky force-pushed the enh/harp/exapp-info-endpoint branch from c6067d2 to f21b930 Compare February 14, 2025 15:20
@kyteinsky
ex-app harp specific env set
15881ab 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
update exapp cache in harp
8618103 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
frp cert copy from harp to exapp
fdeae77 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
ui: add harp daemon config
28127f3 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
move haproxy password out of foldable
88d3567 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
change harp deploy config structure
3ede37f 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
use harp shared key from daemon config
ebab47d 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
wip: improvements to daemon registration modal
9608349 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
ui second draft
f3c1b5c 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
final fixes
e9516ca 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
little ui changes
db1c359 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
connect to exapps through harp
4c5b308 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
compiled assets
dde05a0 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
psalm fixes
6ab9a36 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
exapp always binds to localhost for harp
4d55d0a 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
fix register daemon isHarp option
f6fcd8a 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky kyteinsky force-pushed the enh/harp/exapp-info-endpoint branch from c4bb958 to 5c72779 Compare March 24, 2025 11:20
@kyteinsky
use deploy config from exapp instead of daemon config
46371dc 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky kyteinsky force-pushed the enh/harp/exapp-info-endpoint branch 5 times, most recently from 6a2a52e to 6651e51 Compare March 25, 2025 09:19
@kyteinsky
ci: manual harp deploy test
2f6fc74 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky kyteinsky force-pushed the enh/harp/exapp-info-endpoint branch from 6651e51 to 2f6fc74 Compare March 25, 2025 09:36
@kyteinsky
Copy link
Collaborator Author

kyteinsky commented Mar 25, 2025

/compile

nextcloud-command and others added 2 commits March 25, 2025 09:37
@nextcloud-command
chore(assets): Recompile assets
c2a6430 
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@kyteinsky
feat: default daemon setup checks
d8adb83 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@andrey18106
Copy link
Contributor

andrey18106 commented Mar 25, 2025

@kyteinsky Hey, just tested works fine, except a few errors in Admin settings UI:

  1. Enable HaRP in Register modal not working as expected (wrong template name)
  2. Edit Deploy daemon not working on old daemons (they don't have deploy_config.harp key) - maybe post-process prepare daemons deploy config?
Details

image

@kyteinsky
fix enable harp toggle
39239d0 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
edit deploy daemon and manual-install fixes
9ef202f 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@kyteinsky
Copy link
Collaborator Author

kyteinsky commented Mar 27, 2025 •
edited
Loading

Thanks @andrey18106, please check if those fixed the issues.
for (2.), yes post process fixed it and setupFormConfiguration was also causing some issues.

@kyteinsky
Copy link
Collaborator Author

kyteinsky commented Mar 27, 2025

/compile

@nextcloud-command
chore(assets): Recompile assets
4e64578 
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@bigcat88 bigcat88 mentioned this pull request Mar 30, 2025
Merged
bigcat88
bigcat88 reviewed Mar 30, 2025
View reviewed changes
@kyteinsky
do not fetch exapp twice for apps metadata
bfb3d3d 
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
@oleksandr-nc oleksandr-nc mentioned this pull request Mar 31, 2025
Merged
@oleksandr-nc oleksandr-nc merged commit 3d332ea into main Apr 1, 2025
36 checks passed
@oleksandr-nc oleksandr-nc deleted the enh/harp/exapp-info-endpoint branch April 1, 2025 14:03
@oleksandr-nc oleksandr-nc mentioned this pull request Apr 15, 2025
Merged
This was referenced Apr 24, 2025
Merged
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bigcat88 bigcat88 bigcat88 left review comments

@oleksandr-nc oleksandr-nc oleksandr-nc approved these changes

@andrey18106 andrey18106 Awaiting requested review from andrey18106

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kyteinsky @andrey18106 @bigcat88 @oleksandr-nc @nextcloud-command