Skip to content
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

Synthesis changes #3

Draft
wants to merge 22 commits into
base: upm
Choose a base branch
from
Draft
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
8bc7dbf
Update .gitignore
kristenmontesano Aug 1, 2022
9fd15a5
Update package.json
kristenmontesano Aug 1, 2022
a5c8fa1
fix https fallback to http
kristenmontesano Aug 1, 2022
6cb023a
Create README.md
kristenmontesano Aug 3, 2022
294f967
readme instructions
kristenmontesano Aug 3, 2022
767a7cc
Update README.md
kristenmontesano Aug 3, 2022
1b82317
Update README.md
kristenmontesano Aug 3, 2022
89c412f
Update README.md
kristenmontesano Aug 3, 2022
27437f8
Update README.md
kristenmontesano Aug 3, 2022
fd13a6e
Merge pull request #2 from gosynthschool/km-update-readme
kristenmontesano Aug 3, 2022
963bf9a
Add missing meta file to silence editor errors
ipsquiggle Aug 6, 2022
2cde836
Update ColyseusRequest.cs
kristenmontesano Aug 19, 2022
8cc6aeb
Update Runtime/Scripts/ColyseusRequest.cs
kristenmontesano Aug 19, 2022
ccd18c4
Merge pull request #4 from gosynthschool/km-SECOND-secureProtocol
kristenmontesano Aug 19, 2022
d51315c
Update ColyseusRequest.cs
kristenmontesano Aug 19, 2022
b954762
Merge pull request #5 from gosynthschool/km-remove-retry
kristenmontesano Aug 19, 2022
c6666cd
Add logging to websockets and propagate errors up to the client
ipsquiggle Feb 2, 2023
0adbe81
fix compiler error
ipsquiggle Feb 2, 2023
db8a82d
Fix another compiler error
ipsquiggle Feb 2, 2023
4f0b46f
Revert "Add logging to websockets and propagate errors up to the client"
ipsquiggle Feb 4, 2023
0ac6706
Bump version number for publishing to complete the revert
ipsquiggle Feb 4, 2023
01ec6f7
Merge pull request #6 from gosynthschool/ips/websocket-logging
ipsquiggle Feb 6, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update ColyseusRequest.cs
  • Loading branch information
kristenmontesano committed Aug 19, 2022
commit d51315caef0f2b3843616dacaa64a97e345301c4
24 changes: 4 additions & 20 deletions Runtime/Scripts/ColyseusRequest.cs
Original file line number Diff line number Diff line change
@@ -58,16 +58,8 @@ public async Task<string> Request(string uriMethod, string uriPath, string uriQu
{
if (_serverSettings.useSecureProtocol)
{
//We failed to make this call with a secure protocol, try again
LSLog.LogError($"Failed to make request to {req.url}, trying again!");
try
{
return await Request(uriMethod, uriPath, uriQuery, Token, data);
}
catch
{
LSLog.LogError($"Failed RETRY to make request to {req.url}.");
}
//We failed to make this call with a secure protocol
LSLog.LogError($"Failed to make request to {req.url}.");
}
else
{
@@ -124,16 +116,8 @@ public async Task<string> Request(string uriMethod, string uriPath, Dictionary<s
{
if (_serverSettings.useSecureProtocol)
{
//We failed to make this call with a secure protocol, try again
LSLog.LogError($"Failed to make request to {req.url}, trying again!");
try
{
return await Request(uriMethod, uriPath, options, headers);
}
catch
{
LSLog.LogError($"Failed RETRY to make request to {req.url}.");
}
//We failed to make this call with a secure protocol
LSLog.LogError($"Failed to make request to {req.url}.");
}
else
{