LemmyNet
/
lemmy
3
2
Fork 3
Code Pull Requests 1 Releases Wiki Activity

Dont include community in comment to field (fixes #1446) #174

Merged
dessalines merged 2 commits from no-community-in-comment-to into main 2021-02-24 01:03:09 +00:00
Conversation 1 Commits 2 Files Changed 5 +19 -14
nutomic commented 2021-02-23 18:02:38 +00:00
Owner
Copy Link

This was actually much easier than I thought. Unfortunately I couldnt test if this fixes Pleroma federation, because the local federation setup throws errors when requesting something over HTTPS. I will try to put it up on a test server.

This was actually much easier than I thought. Unfortunately I couldnt test if this fixes Pleroma federation, because the local federation setup throws errors when requesting something over HTTPS. I will try to put it up on a test server.
nutomic added 1 commit 2021-02-23 18:02:39 +00:00
nutomic added 1 commit 2021-02-23 18:35:38 +00:00
nutomic commented 2021-02-23 18:36:09 +00:00
Author
Owner
Copy Link

I threw up the changes on test.lemmy.ml but its not working, lemmy-ui is giving this error:

lemmy-ui_1  | (node:1) UnhandledPromiseRejectionWarning: FetchError: invalid json response body at http://lemmy:8536/api/v2/site? reason: Unexpected endof JSON input
lemmy-ui_1  |     at /app/node_modules/node-fetch/lib/index.js:272:32
lemmy-ui_1  |     at processTicksAndRejections (internal/process/task_queues.js:93:5)
lemmy-ui_1  | (Use `node --trace-warnings ...` to show where the warning was created)
lemmy-ui_1  | (node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
lemmy-ui_1  | (node:1) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handledwill terminate the Node.js process with a non-zero exit code.

And doing curl 127.0.0.1:1235/api/v2/site on the server just hangs forever. Dont know whats going on, but I dont think its related to this PR.

A also added a commit to fix the lemmy_dev.yml playbook, seems like something changed there.

I threw up the changes on test.lemmy.ml but its not working, lemmy-ui is giving this error: ``` lemmy-ui_1 | (node:1) UnhandledPromiseRejectionWarning: FetchError: invalid json response body at http://lemmy:8536/api/v2/site? reason: Unexpected endof JSON input lemmy-ui_1 | at /app/node_modules/node-fetch/lib/index.js:272:32 lemmy-ui_1 | at processTicksAndRejections (internal/process/task_queues.js:93:5) lemmy-ui_1 | (Use `node --trace-warnings ...` to show where the warning was created) lemmy-ui_1 | (node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2) lemmy-ui_1 | (node:1) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handledwill terminate the Node.js process with a non-zero exit code. ``` And doing `curl 127.0.0.1:1235/api/v2/site` on the server just hangs forever. Dont know whats going on, but I dont think its related to this PR. A also added a commit to fix the lemmy_dev.yml playbook, seems like something changed there.
dessalines reviewed 2021-02-24 01:02:36 +00:00
crates/apub/src/objects/post.rs
@ -120,3 +120,2 @@
) -> Result<Post, LemmyError> {
check_object_for_community_or_site_ban(page, context, request_counter).await?;
get_object_from_apub(page, context, expected_domain, request_counter).await
let post: Post = get_object_from_apub(page, context, expected_domain, request_counter).await?;
dessalines commented 2021-02-24 01:02:36 +00:00
Owner
Copy Link

The other ones seem to do this too, so its prolly fine.

The other ones seem to do this too, so its prolly fine.
nutomic commented 2021-02-24 12:58:14 +00:00
Author
Owner
Copy Link

I dont know what you mean :p

I dont know what you mean :p
dessalines merged commit 6499709221 into main 2021-02-24 01:03:09 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: LemmyNet/lemmy#174
No description provided.
Delete Branch "no-community-in-comment-to"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?