MirrorHub/construct
0
0
Fork 0
mirror of https://github.com/matrix-construct/construct synced 2024-06-16 08:58:20 +02:00
Code Issues Releases Wiki Activity

ircd:Ⓜ️:event: Add various size constants replacing magic numbers.

Browse source
This commit is contained in:
Jason Volk 2019-02-18 12:25:00 -08:00
parent bc0e12e31b
commit 0644db459c
6 changed files with 16 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
include/ircd/m/dbs.h
View file

@ -54,12 +54,12 @@ namespace ircd::m::dbs
string_view room_head_key(const mutable_buffer &out, const id::room &, const id::event &);
string_view room_head_key(const string_view &amalgam);
constexpr size_t ROOM_STATE_KEY_MAX_SIZE {id::MAX_SIZE + 256 + 256};
constexpr size_t ROOM_STATE_KEY_MAX_SIZE {id::MAX_SIZE + event::TYPE_MAX_SIZE + event::STATE_KEY_MAX_SIZE};
string_view room_state_key(const mutable_buffer &out, const id::room &, const string_view &type, const string_view &state_key);
string_view room_state_key(const mutable_buffer &out, const id::room &, const string_view &type);
std::pair<string_view, string_view> room_state_key(const string_view &amalgam);
constexpr size_t ROOM_JOINED_KEY_MAX_SIZE {id::MAX_SIZE + 256 + id::MAX_SIZE};
constexpr size_t ROOM_JOINED_KEY_MAX_SIZE {id::MAX_SIZE + event::ORIGIN_MAX_SIZE + id::MAX_SIZE};
string_view room_joined_key(const mutable_buffer &out, const id::room &, const string_view &origin, const id::user &member);
string_view room_joined_key(const mutable_buffer &out, const id::room &, const string_view &origin);
std::pair<string_view, string_view> room_joined_key(const string_view &amalgam);

5
include/ircd/m/event/event.h
View file

@ -92,7 +92,10 @@ struct ircd::m::event
using closure_idx_bool = std::function<bool (const idx &)>;
using closure_iov_mutable = std::function<void (json::iov &)>;
static constexpr size_t MAX_SIZE = 64_KiB;
static constexpr const size_t MAX_SIZE = 64_KiB;
static constexpr const size_t TYPE_MAX_SIZE = 256;
static constexpr const size_t STATE_KEY_MAX_SIZE = 256;
static constexpr const size_t ORIGIN_MAX_SIZE = 256;
static conf::item<size_t> max_size;
static void essential(json::iov &event, const json::iov &content, const closure_iov_mutable &);

2
ircd/m_room.cc
View file

@ -1485,7 +1485,7 @@ ircd::m::room::state::for_each(const types_bool &closure)
const
{
string_view last;
char lastbuf[256]; //TODO: type maxlen
char lastbuf[m::event::TYPE_MAX_SIZE];
if(!present())
{
m::state::for_each(root_id, m::state::iter_bool_closure{[&closure, &last, &lastbuf]

10
modules/client/rooms/state.cc
View file

@ -39,13 +39,13 @@ get__state(client &client,
const resource::request &request,
const room::id &room_id)
{
char type_buf[uint(256 * 1.34 + 1)];
char type_buf[m::event::TYPE_MAX_SIZE];
const string_view &type
{
url::decode(type_buf, request.parv[2])
};
char skey_buf[uint(256 * 1.34 + 1)];
char skey_buf[m::event::STATE_KEY_MAX_SIZE];
const string_view &state_key
{
url::decode(skey_buf, request.parv[3])
@ -53,7 +53,7 @@ get__state(client &client,
// (non-standard) Allow an event_id to be passed in the query string
// for reference framing.
char evid_buf[uint(256 * 1.34 + 1)];
char evid_buf[m::id::MAX_SIZE];
const string_view &event_id
{
url::decode(evid_buf, request.query["event_id"])
@ -73,13 +73,13 @@ put__state(client &client,
const resource::request &request,
const room::id &room_id)
{
char type_buf[uint(256 * 1.34 + 1)];
char type_buf[m::event::TYPE_MAX_SIZE];
const string_view &type
{
url::decode(type_buf, request.parv[2])
};
char skey_buf[uint(256 * 1.34 + 1)];
char skey_buf[m::event::STATE_KEY_MAX_SIZE];
const string_view &state_key
{
url::decode(skey_buf, request.parv[3])

4
modules/client/user/account_data.cc
View file

@ -34,7 +34,7 @@ put__account_data(client &client,
"type path parameter required"
};
char typebuf[256];
char typebuf[m::event::TYPE_MAX_SIZE];
const string_view type
{
url::decode(typebuf, request.parv[2])
@ -67,7 +67,7 @@ get__account_data(client &client,
"type path parameter required"
};
char typebuf[256];
char typebuf[m::event::TYPE_MAX_SIZE];
const string_view type
{
url::decode(typebuf, request.parv[2])

4
modules/client/user/rooms.cc
View file

@ -121,7 +121,7 @@ put__account_data(client &client,
"type path parameter required"
};
char typebuf[256];
char typebuf[m::event::TYPE_MAX_SIZE];
const auto &type
{
url::decode(typebuf, request.parv[4])
@ -156,7 +156,7 @@ get__account_data(client &client,
"type path parameter required"
};
char typebuf[256];
char typebuf[m::event::TYPE_MAX_SIZE];
const auto &type
{
url::decode(typebuf, request.parv[4])