banquise/pelican-panel-mirror
3
0
Fork 1
mirror of https://github.com/pelican-dev/panel.git synced 2025-05-20 20:24:44 +02:00
Code Issues Packages Projects Releases Wiki Activity

Update token rotation

Browse Source
This commit is contained in:
notCharles 2024-06-07 17:38:58 -04:00
parent 6c2d0a2d50
commit 7ee52affb2
1 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
app/Services/Nodes/NodeUpdateService.php
View File

@ -27,15 +27,22 @@ class NodeUpdateService
*/ */
public function handle(Node $node, array $data, bool $resetToken = false): Node public function handle(Node $node, array $data, bool $resetToken = false): Node
{ {
$data['id'] = $node->id;
if ($resetToken) { if ($resetToken) {
$data['daemon_token'] = Str::random(Node::DAEMON_TOKEN_LENGTH); $data['daemon_token'] = Str::random(Node::DAEMON_TOKEN_LENGTH);
$data['daemon_token_id'] = Str::random(Node::DAEMON_TOKEN_ID_LENGTH); $data['daemon_token_id'] = Str::random(Node::DAEMON_TOKEN_ID_LENGTH);
} }
[$node, $exception] = $this->connection->transaction(function () use ($data, $node) { [$updated, $exception] = $this->connection->transaction(function () use ($data, $node) {
$node->forceFill($data)->save(); /** @var \App\Models\Node $updated */
$updated = $node->replicate();
$updated->exists = true;
$updated->forceFill($data)->save();
try { try {
$this->configurationRepository->setNode($node)->update($node); $node->fqdn = $updated->fqdn;
$this->configurationRepository->setNode($node)->update($updated);
} catch (DaemonConnectionException $exception) { } catch (DaemonConnectionException $exception) {
logger()->warning($exception, ['node_id' => $node->id]); logger()->warning($exception, ['node_id' => $node->id]);
@ -45,16 +52,16 @@ class NodeUpdateService
// //
// This avoids issues with proxies such as Cloudflare which will see daemon as offline and then // This avoids issues with proxies such as Cloudflare which will see daemon as offline and then
// inject their own response pages, causing this logic to get fucked up. // inject their own response pages, causing this logic to get fucked up.
return [$node, true]; return [$updated, true];
} }
return [$node, false]; return [$updated, false];
}); });
if ($exception) { if ($exception) {
throw new ConfigurationNotPersistedException(trans('exceptions.node.daemon_off_config_updated')); throw new ConfigurationNotPersistedException(trans('exceptions.node.daemon_off_config_updated'));
} }
return $node; return $updated;
} }
} }