-
Notifications
You must be signed in to change notification settings - Fork 786
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
Next js prototype #3896
Next js prototype #3896
Changes from all commits
2eab14d
e970d16
4eaa65d
14b340c
0c0fa94
defe9b5
5f806c7
0ec39aa
18e8b85
18fd79b
adaac7d
5b78f50
5f4627c
d5eb2f1
d1d3719
a9a9a5e
08575cf
9127ffe
e98d3b9
7bf1e62
874abbf
fecf3dc
47624ba
2a6b80f
300302f
c0c19ca
4d32b37
17e40d6
e5f21f7
9c701cd
c40035e
a6e5560
a8152ed
1a566a9
2a9e5de
edf45aa
d0d9713
06284f1
4d918b7
47c48af
f632afd
670a651
36e3093
bf98349
34cd423
89c8830
426257f
73717d0
c005f54
773ea0a
f10c851
eb79b0c
d1826d5
ade1093
74f41e4
a479eb8
5588a59
c806301
b920bee
d580f92
c8d8663
7194b88
2424d12
9bb7457
4497149
1660f02
74d5db5
86d5047
32a7cd9
0408733
5e6509d
abbdf60
f90c0f5
d4f9b01
b8cd608
4b3ee15
d502133
0aabc4e
05aabb2
4fa163d
9332a74
0aeba18
864456c
3768196
a081163
020f2e3
7fdbdef
9a1b755
10877e1
da57a2d
86bbda1
3ce595e
435c285
36ace92
d17defa
e19f6aa
b8dd716
6daf1c7
41a965a
bc76664
6df981a
ac17b2b
c0c50a0
b332000
599e923
b58673a
a5a20d6
6ab1762
13a0fc8
cd11f28
a0f7e03
e9074e4
cd10c44
d782e43
7ab7286
c1327e9
4536a6c
77262b7
5a41863
8861a96
49db21d
4b1d301
4e74ef2
58cd5b3
98bf825
af9c4b2
7104b41
0e0fa9a
301a41e
0280ea5
7842782
1823b50
1943efc
76de8a6
dfb5493
fe361d7
048dab2
b878934
49cbad4
9a56184
50b26ea
a2a1821
93f2fba
578fb53
992fb83
9afb808
db2e4d2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -70,6 +70,7 @@ fi | |
# We can set main paths after we know the home dir | ||
ISLAND_PATH="$monkey_home/monkey/monkey_island" | ||
MONGO_PATH="$ISLAND_PATH/bin/mongodb" | ||
NODE_SERVER_PATH="$ISLAND_PATH/bin/node" | ||
ISLAND_BINARIES_PATH="$ISLAND_PATH/cc/binaries" | ||
INFECTION_MONKEY_DIR="$monkey_home/monkey/infection_monkey" | ||
MONKEY_BIN_DIR="$INFECTION_MONKEY_DIR/bin" | ||
|
@@ -191,6 +192,9 @@ log_message "Generating certificate" | |
chmod u+x "${ISLAND_PATH}"/linux/create_certificate.sh | ||
"${ISLAND_PATH}"/linux/create_certificate.sh ${ISLAND_PATH}/cc | ||
|
||
# Install node server | ||
"${ISLAND_PATH}"/linux/install_node.sh "${NODE_SERVER_PATH}" || handle_error | ||
|
||
# Update node | ||
if ! exists npm; then | ||
log_message "Installing nodejs" | ||
|
@@ -235,7 +239,7 @@ pushd "$ISLAND_PATH/cc/ui" || handle_error | |
npm ci | ||
|
||
log_message "Generating front end" | ||
npm run dev | ||
npm run build | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does the current UI support a There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure what value the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let me rephrase. This is generating the frontend for the old UI ( |
||
popd || handle_error | ||
|
||
# Making dir for binaries | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll need to devise a way to expose this port through docker. Either that or provide documentation on how to expose the configured port when running the container
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAIK
--network=host
already exposes the port.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That may be correct. I tried to test out this feature in docker and couldn't access the UI. I realized that modifying the server port is only applicable to the old UI but I still couldn't get it to work. In fact, if I modify it, the UI is not available at the original port (5000) or the new one