Giter Club home page Giter Club logo

Comments (13)

4kangjc avatar 4kangjc commented on May 5, 2024

这个Remove有点丑陋...

bool HttpHeaders::Remove(std::string_view key) noexcept {
if (!TryGet(key)) {
return false;
}
header_idx_.clear();
std::size_t i = 0;
auto it = fields_.begin();
while (it != fields_.end()) {
if (internal::detail::hash_map::CaseInsensitiveEqualTo()(it->first, key)) {
it = fields_.erase(it);
} else {
if (!header_idx_.contains(it->first)) {
header_idx_[it->first] = i;
}
++i;
++it;
}
}
// We don't remove the key/value in owning_strs_2_ because it costs.
return true;
}

如果用internal::CaseInsensitiveHashMap<std::string_view, std::string_view>, 直接erase(key)就行?

from flare.

4kangjc avatar 4kangjc commented on May 5, 2024

hmm,deque我好像知道为什么了,vector会扩容,导致之前的string被释放

from flare.

0x804d8000 avatar 0x804d8000 commented on May 5, 2024

是说把headers存成map吗?这个不行,headers里面可以有重复的key,解析/序列化的时候都需要保留

from flare.

4kangjc avatar 4kangjc commented on May 5, 2024

是说把headers存成map吗?这个不行,headers里面可以有重复的key,解析/序列化的时候都需要保留

重复的key吗?有点奇怪,那Set函数的定义是修改最后一个key?

from flare.

0x804d8000 avatar 0x804d8000 commented on May 5, 2024

对于不存在或者只出现了一次的,set行为是明确的,如果已经重复了,flare的实现的话行为没定义(而且也不好定义)。这种情况下一般是多次调用add来添加header

from flare.

4kangjc avatar 4kangjc commented on May 5, 2024

对于不存在或者只出现了一次的,set行为是明确的,如果已经重复了,flare的实现的话行为没定义(而且也不好定义)。这种情况下一般是多次调用add来添加header

呃,问题是用户在调用set的时候不一定知道有没有重复?

from flare.

0x804d8000 avatar 0x804d8000 commented on May 5, 2024

理论上有这个可能,但是实际使用中其实有两类case:

  1. 接收到HTTP包,这时候不知道都有哪些header,但是这种case只需要解析,通常不会需要修改别人发来的数据
  2. 构造HTTP包,这时候用户清楚自己填充过哪些header

唯一的风险是作为代理转发,但是这种情况下一般既不读也不写,所以也没问题。

又读又写的case应该很少见,而且在符合rfc的范围之内,重复key的set的行为怎么定义多少都是有问题的。


这儿确实有个已知问题,就是HttpHeaders现在对于读/写共用了一套实现,所以两种情况的性能都不太好。但是想分开做的话可能得做成HttpHeaders + HttpHeadersBuilder之类的,接口比较难看,另外考虑到一般这点性能也不是很关键(不然应该直接用RPC了),所以综合考虑也就保持这样了。

from flare.

4kangjc avatar 4kangjc commented on May 5, 2024

理论上有这个可能,但是实际使用中其实有两类case:

  1. 接收到HTTP包,这时候不知道都有哪些header,但是这种case只需要解析,通常不会需要修改别人发来的数据
  2. 构造HTTP包,这时候用户清楚自己填充过哪些header

唯一的风险是作为代理转发,但是这种情况下一般既不读也不写,所以也没问题。

又读又写的case应该很少见,而且在符合rfc的范围之内,重复key的set的行为怎么定义多少都是有问题的。


这儿确实有个已知问题,就是HttpHeaders现在对于读/写共用了一套实现,所以两种情况的性能都不太好。但是想分开做的话可能得做成HttpHeaders + HttpHeadersBuilder之类的,接口比较难看,另外考虑到一般这点性能也不是很关键(不然应该直接用RPC了),所以综合考虑也就保持这样了。

嗯好,我在想是否可以把相同key的value合成同一个? 一般http报文中也是在一起?而不是显示两个key-value键值对?

from flare.

0x804d8000 avatar 0x804d8000 commented on May 5, 2024

不,去除重复key这个行为是non-conforming的

see also https://stackoverflow.com/questions/4371328/are-duplicate-http-response-headers-acceptable

from flare.

4kangjc avatar 4kangjc commented on May 5, 2024

不,去除重复key这个行为是non-conforming的

see also https://stackoverflow.com/questions/4371328/are-duplicate-http-response-headers-acceptable

哦哦,我知道了

from flare.

4kangjc avatar 4kangjc commented on May 5, 2024

有点遗憾的是Remove的时间复杂度有点高

from flare.

0x804d8000 avatar 0x804d8000 commented on May 5, 2024

对。这个主要是对作为代理的case性能有影响,普通的HTTP服务端或者客户端应该不涉及remove的需求

from flare.

4kangjc avatar 4kangjc commented on May 5, 2024

对。这个主要是对作为代理的case性能有影响,普通的HTTP服务端或者客户端应该不涉及remove的需求

嗯好

from flare.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.