Nosial/socialbox-php
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Made message signing in Cryptography use SHA512 as the message content for... #1

Closed
netkas wants to merge 421 commits from master into dev
pull from: master
merge into: Nosial:dev
Conversation 3 Commits 421 Files changed 226 +4 -4
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit d2119df824 - Show all commits

4
src/Socialbox/Objects/ClientRequest.php
View file

@ -73,12 +73,12 @@
return $this->requestBody; return $this->requestBody;
} }
public function getClientName(): string public function getClientName(): ?string
{ {
return $this->clientName; return $this->clientName;
} }
public function getClientVersion(): string public function getClientVersion(): ?string
{ {
return $this->clientVersion; return $this->clientVersion;
} }

4
src/Socialbox/Socialbox.php
View file

@ -81,14 +81,14 @@
private static function handleInitiateSession(ClientRequest $clientRequest): void private static function handleInitiateSession(ClientRequest $clientRequest): void
{ {
if(!isset($requestHeaders[StandardHeaders::CLIENT_NAME->value])) if(!$clientRequest->getClientName())
{ {
http_response_code(400); http_response_code(400);
print('Missing required header: ' . StandardHeaders::CLIENT_NAME->value); print('Missing required header: ' . StandardHeaders::CLIENT_NAME->value);
return; return;
} }
if(!isset($requestHeaders[StandardHeaders::CLIENT_VERSION->value])) if(!$clientRequest->getClientVersion())
{ {
http_response_code(400); http_response_code(400);
print('Missing required header: ' . StandardHeaders::CLIENT_VERSION->value); print('Missing required header: ' . StandardHeaders::CLIENT_VERSION->value);