yt_dlp/extractor/facebook.py

@@ -52,7 +52,7 @@ class FacebookIE(InfoExtractor):
                             )\?(?:.*?)(?:v|video_id|story_fbid)=|
                             [^/]+/videos/(?:[^/]+/)?|
                             [^/]+/posts/|
-                            groups/[^/]+/(?:permalink|posts)/|
+                            groups/[^/]+/permalink/|
                             watchparty/
                         )|
                     facebook:
@@ -614,8 +614,8 @@ class FacebookIE(InfoExtractor):
                     ..., 'comet_sections', 'content', 'story', (None, 'attached_story'), 'attachments',
                     ..., ('styles', 'throwbackStyles', 'style_type_renderer'), (None, ...), 'attachment'), expected_type=dict) or []
                 for attachment in attachments:
-                    ns = traverse_obj(attachment, ('all_subattachments', 'nodes', ..., {dict}),
-                                      (..., 'attachments', ..., 'styles', 'attachment', {dict}))
+                    ns = (try_get(attachment, lambda x: x['all_subattachments']['nodes'], list)
+                          or traverse_obj(attachment, (..., 'attachments', ..., 'styles', 'attachment'), expected_type=dict) or [])
                     for n in ns:
                         parse_attachment(n)
                     parse_attachment(attachment)
@@ -638,7 +638,7 @@ class FacebookIE(InfoExtractor):
                 if len(entries) > 1:
                     return self.playlist_result(entries, video_id)
 
-                video_info = entries[0] if entries else {'id': video_id}
+                video_info = entries[0] if len(entries) > 0 else {'id': video_id}
                 webpage_info = extract_metadata(webpage)
                 # honor precise duration in video info
                 if video_info.get('duration'):