GitShow/facebook/fbthrift
facebook

fbthrift

Facebook's branch of Apache Thrift, including a new C++ server.

by facebook
Star on GitHubFork

C++

2.7k stars637 forks1.8k contributorsActive · 37m agoSince 2013v2020.08.24.00Apache-2.0

Meet the team

See all 1849 on GitHub →
Mizuchi
Mizuchi1.6k contributions
yfeldblum
yfeldblum1.2k contributions
iahs
iahs1.1k contributions
echistyakov
echistyakov1.1k contributions
thedavekwon
thedavekwon1.0k contributions
vitaut
vitaut824 contributions
praihan
praihan762 contributions
ahilger
ahilger720 contributions

Languages

View on GitHub →
C++61.3%
Python9.8%
Thrift9.7%
Java8.5%
Hack2.8%
Go2.2%
Other5.7%

Commit activity

Last 12 weeks · 1133 commits

Full graph →

Community health

4 of 6 standards met

Community profile →
75
✓README✓License✓Contributing✓Code of Conduct○Issue Template○PR Template

Recent PRs & issues

Active · Last activity 37m ago
See all on GitHub →
durvesh1992
Fix duplicated-word and misspelling typos in comments and docsOpenPR

Summary Fixes a batch of high-confidence typos in comments, documentation, and one flag-description string. These are inspection-only changes — no logic or behavior is modified, and no generated files are touched. Found via a duplicated-word + common-misspelling sweep over non-generated sources. Care was taken to avoid false positives such as legitimate C++/Thrift tokens (e.g. , ). Changes Duplicated words Misspellings Notes 37 files changed, 38 lines, all comments/docs/strings. The same and strings also appear in generated / fixtures; those were intentionally not edited since they are regenerated from the (now-fixed) sources. C++ was not built/tested for this change as it contains no compilable code changes.

durvesh1992 · 2d ago
xiaozhuai
Fix compatibility with fmt 12.2.0OpenPR

Problem fmtlib/fmt deprecated `#include #include `, which is the standard header in fmt 12.2.0+.

xiaozhuai · 1w ago

Recent fixes

View closed PRs →
daeho-ro
Add missing <cstring> includes for std::mem*/std::str*MergedPR

Add missing includes for std::mem/std::str These files call std::memcpy/std::memset/std::strlen etc. but rely on being pulled in transitively via other standard headers. Newer libstdc++ (GCC 13) no longer provides it that way, so they fail to compile: error: 'memcpy' is not a member of 'std'; did you mean 'wmemcpy'? Include directly in each affected translation unit (include-what-you-use).

daeho-ro · 1w ago
zhztheplayer
Update FBThriftCppLibrary.cmake to use PROJECT_SOURCE_DIR and PROJECT_BINARY_DIRMergedPR

The Bug When the CMake project A, which is using , is imported by another parent CMake project B by , project B may not build because the generated CPP sources refer to header files that don't exist. E.g., While the generated header is under . The Fix The PR is a general fix for the issue by turning / to / , to make the generated sources portable for any parent CMake projects' use. This fixes Velox bug https://github.com/facebookincubator/velox/issues/17853.

zhztheplayer · 1w ago
digiscrypt
Add basic libFuzzer target for protocol parsingMergedPR

Hi, This adds a basic libFuzzer target for the C++ protocol layer. It exercises BinaryProtocol and CompactProtocol by feeding arbitrary input through message and struct parsing paths, with simple limits to avoid runaway recursion and very large allocations. The goal is to cover core deserialization logic and make OSS-Fuzz integration easier. Let me know if you want anything changed.

digiscrypt · 2mo ago
Structured data for AI agents

Repository: facebook/fbthrift. Description: Facebook's branch of Apache Thrift, including a new C++ server. Stars: 2692, Forks: 637. Primary language: C++. Languages: C++ (61.3%), Python (9.8%), Thrift (9.7%), Java (8.5%), Hack (2.8%). License: Apache-2.0. Latest release: v2020.08.24.00 (5y ago). Open PRs: 7, open issues: 45. Last activity: 37m ago. Community health: 75%. Top contributors: Mizuchi, yfeldblum, iahs, echistyakov, thedavekwon, vitaut, praihan, ahilger, hchokshi, aristidisp and others.

·@ofershap

Replace github.com with gitshow.dev