Update HttpService.yaml with HTTP methods, explanations, and resources.

[sammygrey]

Changes

Updated HttpService.yaml to provide a short description of each of the HTTP methods, if they are permitted via Roblox scripts, and Mozilla resources for each. This is a duplicate of pull request #966 as the checks did not seem to recognize the changes made to tools/checks/utils/allowedHttpLinks.txt to allow the links to Mozilla resources on each of the HTTP methods.

The only method not currently permitted by Roblox is the CONNECT method which creates a secure information tunnel between the requested and originating server. Makes sense why this isn't allowed, but its useful information to include nonetheless.

[sammygrey]

I literally already did this, I guess ill have to open a pull request with the only change being to tools/checks/utils/allowedHttpLinks.txt

[IgnisRBX]

I took this PR as an opportunity to clean up the overall page, so you'll see a bunch of formatting and minor tweaks in my commit.

In regards to your addition, though, you had both a "Supported" and "Safe" column in the table, but I think you meant to only have "Safe", so I deleted the entire "Supported" column (which was empty and had nothing in any row cell). However, this might not have been your intent, since now some request methods like POST are marked as "safe" but the Mozilla reference marks them as unsafe. Can you please confirm all of the entries are marked correctly as "safe" or "not safe"? If you need to add back the "Supported" column, go ahead, just make sure everything lines up correctly. Thanks!

[IgnisRBX]

Following up, I see your comment that only CONNECT is not supported. So, I prefer that is omitted from the added table, since it cannot be used in Roblox context. And I changed the header of that new section to "Supported HTTP headers" to clarify that the table includes only supported methods (vs. having a column for "Supported" in the table with only CONNECT marked as unsupported).

[sammygrey]

I took this PR as an opportunity to clean up the overall page, so you'll see a bunch of formatting and minor tweaks in my commit.

In regards to your addition, though, you had both a "Supported" and "Safe" column in the table, but I think you meant to only have "Safe", so I deleted the entire "Supported" column (which was empty and had nothing in any row cell). However, this might not have been your intent, since now some request methods like POST are marked as "safe" but the Mozilla reference marks them as unsafe. Can you please confirm all of the entries are marked correctly as "safe" or "not safe"? If you need to add back the "Supported" column, go ahead, just make sure everything lines up correctly. Thanks!

Thats my bad, looks like I copied the sections of that collumn as a template for the markdown, but never actually changed the values to their correct counterparts. Should be fixed now.

[sammygrey]

I took this PR as an opportunity to clean up the overall page, so you'll see a bunch of formatting and minor tweaks in my commit.

In regards to your addition, though, you had both a "Supported" and "Safe" column in the table, but I think you meant to only have "Safe", so I deleted the entire "Supported" column (which was empty and had nothing in any row cell). However, this might not have been your intent, since now some request methods like POST are marked as "safe" but the Mozilla reference marks them as unsafe. Can you please confirm all of the entries are marked correctly as "safe" or "not safe"? If you need to add back the "Supported" column, go ahead, just make sure everything lines up correctly. Thanks!

Replying to this again because I had to remove some of the changes you made as I'm lacking the permissions to edit code_sample sections. You should have this permission however, so I made a gist that's just the file before the code_sample removals. You should be able to easily merge this if you so choose.

[IgnisRBX]

Are there further edits needed for the code samples? If so, can you indicate which samples (by name) need edits, and what those edits are? Thanks!

[sammygrey]

Are there further edits needed for the code samples? If so, can you indicate which samples (by name) need edits, and what those edits are? Thanks!

I believe the change you made that couldn't be implemented was just removing the "International-Space-Station" code sample from the areas that included it. It threw an error with checks because I don't have adequate permissions to change code samples. I don't believe anything else needs editing, that was the only part that failed checks.