text:
Reporting security issues

Where/How do we report security issues for this repository?

Please direct to security@cloudposse.com

repo: cloudposse/github-commenter


text:
RabbitMQ prometheus endpoint return status code 406

Current implementation performs HTTP request with Accept header including quality factor (q parameter) and extensions (see code).
RabbitMQ prometheus endpoint returns HTTP status code 406 Not Acceptable (see RabbitMQ issue) with current Accept header. The problem is related to the version=0.0.4 extension of the text/plain media type, removing the version=0.0.4 from the header value resolves the issue.
Is there any specific reason to have the version=0.0.4 extension specified for the text/plain media type?
Would it be acceptable to have it removed? (I can provide PR)
Thanks,
RJB

FWIW this is already fixed in RabbitMQ via this commit.

Yes, the issue can be closed

source: gharchive/issue

authors: lukebakken rjbaucells

repo: cloudposse/prometheus-to-cloudwatch

license_type: permissive

license_source: github-api


text:
Use non deprecated tag keys PR causes bug

Found a bug? Maybe our Slack Community can help.

Describe the Bug
Existing ASG with tags have this error The argument "tag.4.value" is required, but no definition was found. after upgrading to version 0.31.1. Can only do terraform apply if tags are removed.
(Changes from this PR https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/pull/95)
Expected Behavior
Expected existing resources to have no change when upgrading from 0.31.0 to 0.31.1
Steps to Reproduce
Steps to reproduce the behavior:

Create ASG resources from module cloudposse/ec2-autoscale-group/aws version 0.31.0
Updating module version to 0.31.1
Run terraform plan
See error

Error: Missing required argument
│
│   with module.go_translate.module.autoscale_group.aws_autoscaling_group.default[0],
│   on .terraform/modules/go_translate.autoscale_group/main.tf line 144, in resource "aws_autoscaling_group" "default":
│  144: resource "aws_autoscaling_group" "default" {
│
│ The argument "tag.4.value" is required, but no definition was found.
╵

Anything that will help us triage the bug will help. Here are some ideas:

OS: [e.g. Linux, OSX, WSL, etc] OSX
Version [e.g. 10.15] 12.2

How could this be fixed by the terraform?
If it cannot be fixed, is it better to document the path you described in order to upgrade from the old version to the new version?

How could this be fixed by the terraform?
If it cannot be fixed, is it better to document the path you described in order to upgrade from the old version to the new version?

This can be fixed by remove empty/null values from tags https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/pull/101/files#diff-dc46acf24afd63ef8c556b77c126ccc6e578bc87e3aa09a931f33d9bf2532fbbR142-R145

Fixed by https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/pull/101

authors: linhkikuchi nitrocode


text:
Allow defining a certificate

Describe the Feature
Allow defining a certificate for the domain.
When I setup a name for the ES or Kibana services like this:
domain_hostname_enabled = true
elasticsearch_subdomain_name = "elasticsearch"
kibana_hostname_enabled = true
kibana_subdomain_name   = "kibana"
dns_zone_id             = aws_route53_zone.hosted_zone.zone_id
The records are correctly setup in Route53 and I can see kibana.my.domain.com pointing to the correct URL (the long ugly one). However when I try to navigate to the service using this pretty name I get an error message in the browser because it uses the AWS ugly certificate. I can't find a way to set my own certificate.
Expected Behavior
I would like to setup a certificate for my.domain.com from the ACM. Notice the AWS UI has an option exactly for that purpose.
Use Case
Should allow setting up the certificate for my alternate name.
Describe Ideal Solution
Pass the certificate ARN as a variables and it will be registered in the domain.
Alternatives Considered
Setting up an ELB... but the ES already has the functionality.
Additional Context
Screenshot shows the ES certificate which won't work with my kibana.my.domain.com name

Requires https://github.com/hashicorp/terraform-provider-aws/issues/16059 ?

@Spritekin @christiangjengedal we now have https://github.com/cloudposse/terraform-aws-elasticsearch#input_custom_endpoint_certificate_arn which I believe would satisfy this constraint. I'm going to close out. Please reopen if I'm mistaken.

authors: Gowiem Spritekin christiangjengedal

repo: cloudposse/terraform-aws-elasticsearch


text:
Create an IRSA without an IAM policy

Found a bug? Maybe our Slack Community can help.

Describe the Bug
You may want an IRSA for best practices without an IAM policy to further reduce blast radius so you can have a pod not use the eks node worker role by default.
If you enable an iam role without supplying policy statements, you receive this bug
╷
│ Error: creating IAM Policy
|
| MalformedPolicyDocument: Syntax errors in policy.
│
│   with module.datadog_agent.module.eks_iam_role.aws_iam_policy.service_account[0],
│   on .terraform/modules/datadog_agent.eks_iam_role/main.tf line 85, in resource "aws_iam_policy" "service_account":
│   85: resource "aws_iam_policy" "service_account" {

Expected Behavior
Enable the IAM role without creating an iam policy resource
Additional context
This may need to be added directly into the upstream module and then bumped here

https://github.com/cloudposse/terraform-aws-iam-policy/issues/23

Closing as wontfix because you can always supply a policy, even if it is
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Effect": "Deny",
            "Action": "*",
            "Resource": "*"
        }
    ]
}

@Nuru Why does one need to provide this Deny policy if this can be solved simply with a iam_policy_enabled and pass it down to iam_policy module?

authors: Nuru dennislapchenko nitrocode

repo: cloudposse/terraform-aws-helm-release


text:
Add an optional list of IP CIDRs which can access the S3 website.

what

We have a need to restrict access to the S3 website.  This change allows adding the aws:SourceIp condition to the IAM bucket policy via the new optional trusted_ips variable.

why

S3 website are normally public, but we have a need to not allow the work access to the contents.  Source IP restrictions help to solve that.

/terratest

source: gharchive/pull-request

authors: hans-d razorsedge


text:
Decoupling various resources from context module

Removing dependency on some resources (such as IAM role, policy, and
bucket replication rule) from the label module.
Also added replication to the complete example.

Why is this change being made ? What problem is this solving ?

Why is this change being made? What problem is this solving?

Created a new issue to describe it better: https://github.com/cloudposse/terraform-aws-tfstate-backend/issues/109

Thank you for the contribution but it's unclear what this is solving for. Updating the example makes sense but the rest of the changes do not. Could you explain more? This is one reason why we request filling out the PR template.

authors: nitrocode thiagoalmeidasa


text:
Make autosuggest trigger and options list less coupled

Description
The autosuggest component is used internally by the property filter where the coming use case requires the autosuggest to be better abstract allowing for any content in the dropdown. The PR factors out a new internal element - autosuggest options list to reduce coupling between the internal input and list-options elements.
How has this been tested?
Existing tests were updated to account for structural changes.
New tests added to cover the tab traps for recovery link.
Documentation changes
[Do the changes include any API documentation changes?]

[ ] Yes, this change contains documentation changes.
[x] No.

Related Links
[Attach any related links/pull request for this change]

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

[ ] Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
[ ] Changes do not include unsupported browser features, see CONTRIBUTING.md.
[ ] Changes were manually tested for accessibility, see accessibility guidelines.

Security

[ ] If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

[ ] Changes are covered with new/existing unit tests?
[ ] Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #114 (f4ac39c) into main (7ddf8e2) will increase coverage by 0.01%.
The diff coverage is 97.56%.

@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
+ Coverage   92.37%   92.38%   +0.01%
==========================================
  Files         535      535
  Lines       15464    15478      +14
  Branches     4255     4257       +2
==========================================
+ Hits        14285    14300      +15
- Misses       1095     1096       +1
+ Partials       84       82       -2

Impacted Files
Coverage Δ

src/autosuggest/options-list.tsx
97.36% <97.36%> (ø)

src/autosuggest/internal.tsx
93.45% <100.00%> (-0.61%)
:arrow_down:

src/link/internal.tsx
94.91% <0.00%> (+1.47%)
:arrow_up:

src/autosuggest/virtual-list.tsx
90.90% <0.00%> (+3.03%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

authors: codecov-commenter pan-kot

repo: cloudscape-design/components


text:
UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v3.17 to before v5.4.126",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=31434f496abb9f3410b10f541462fe58613dd3ad",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=0057ecef9f324007c0ba5fcca4ddd131178ce78b"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "31434f496abb9f3410b10f541462fe58613dd3ad",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "0057ecef9f324007c0ba5fcca4ddd131178ce78b",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "NFS: Fix a potential NULL dereference in nfs_get_client()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.126 by commit 0057ecef9f324007c0ba5fcca4ddd131178ce78b, it was introduced in version v3.17 by commit 31434f496abb9f3410b10f541462fe58613dd3ad. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000905

repo: cloudsecurityalliance/security-database


text:
不使用stomp协议的skynet rabbitmq模块

多数固有的rabbitmq队列并不支持stomp协议，希望能合入一个ampq协议的rabbitmq工具。
建议参考：
https://github.com/gsdenys/amqp-client
经测试、修改后可以正常使用，已知消息少的时候会出现异常、需要写入权限才能消费

我认为不宜再增加现有 skynet 仓库的维护负担。任何库的引入，都意味着有责任保证它没有问题。所以，建议把链接放在 wiki 里面。

我写过个 可以参考下
https://github.com/cloudfreexiao/skynet_rabbitmq

上面那个库我的个人意见是：应该把位操作、64bit 整数运算、解包函数这些，用新的 lua 内置语法重新实现以下。

我写过个 可以参考下纯 lua 实现 如果需要修改 应该问题也不大 能用 额 我好像看错了问题 )
https://github.com/cloudfreexiao/skynet_rabbitmq

哈哈 有拜读过这个实现，现有的队列不支持stomp，撕逼无果之后选择换了个方案

上面那个库我的个人意见是：应该把位操作、64bit 整数运算、解包函数这些，用新的 lua 内置语法重新实现以下。

我加了一个bit.lua 简单跑通了这个解决方案，没有做比较好的优化，没好意思上传。稍后我试试放wiki上

你贴的 链接 可以拿来改改 就是 代码多了点

string.unpack 可以替代掉反复的 string.byte 解码。
bit 库现在已经可以有内建的 bit 运算解决。
现在整数已经支持 64bit 不再需要刻意考虑，但整数除法应该考虑使用 // 。

authors: cloudfreexiao cloudwu fattyark

repo: cloudwu/skynet

license: MIT


text:
Update scalafmt-core to 3.8.1

About this PR
📦 Updates org.scalameta:scalafmt-core from 2.7.5 to 3.8.1 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #710.

Superseded by #718.


text:
Problems adding datetime metadata to a file or dataset

Describe the bug
I am unable to add any "Date and Time" metadata values. It appears the datepicker is never initialized, and the field is just a text input? Entering a value manually doesn't appear to work either, as the resulting term appears to be blank.
Is anyone else seeing this behavior as well?
To Reproduce
Steps to reproduce the behavior:

Add a new metadata terms representing a Datetime, call it date
Upload/navigate to a file (in a dataset in a space)
Attempt to attach metadata to a file: date: 2020-10-02

Note that the datepicker doesn't display

Manually enter a value in an acceptable format (e.g. 2020-10-02) and click Submit

Note that the metadata does not appear automatically

Refresh the page and scroll down to your new metadata

Expected behavior
The entered date should be displayed under the file's metadata as date: 2020-10-02
Actual behavior
A blank box is shown where this new metadata should be
Screenshots
If applicable, add screenshots to help explain your problem.
Input focus should display a Datepicker:

Entered value should persist as metadata:

Desktop (please complete the following information):

OS: iOS
Browser: Chrome
Version: 85

Additional context
This behavior also happens with the built-in "Date and Time" metadata term

@lmarini
I assigned this one to me.

A few things:
I notice that some extractors post datetime metadata correctly (such as ncsa.clamav). So I checked and the issue is definitely with the form. With datetime, the metadata send in as part of the post request is empty.
This seems to have something to do with #field-value not working when the datepicker is used. I saw a link to a stackoverflow in the code, looking into that.

repo: clowder-framework/clowder

license: NCSA


text:
Setup instructions: MongoDB no longer includes mongorestore

As per documentation on this page:
https://docs.mongodb.com/database-tools/mongorestore/#installation
mongorestore is now part of MongoDB Database Tools, which are not included with MongoDB Server. This should be reflected in the main ReadMe.

Please feel free to fix the instructions!

Will do - this is partly logged as a mental note while I figure out how to get this set up / working! :)

repo: clowdr-app/clowdr-web-app


text:
[RFC] Functional API

The current OOP API looks a bit flawed IMO.
This ticket serves as a discussion basis whether it makes sense to use plain functions instead.
Current API:
$promise = React\Promise\resolve(123);
$blocker = new Clue\React\Block\Blocker($loop);

$value = $blocker->awaitOne($promise);

Suggested API (prototype):
$promise = React\Promise\resolve(123);

$value = Clue\React\Block\awaitOne($promise, $loop);

Some random thoughts:

The Promise-API already follows a functional pattern, so this feels natural
A similar functional API worked out quite well in a related project: clue/promise-timeout.
The $loop argument could eventually become obsolete (i.e. optional) once React singularity is being considered.
DI for the Blocker is hardly useful, so I'm not really seeing any benefit whatsoever

Also, regarding functional composition: Quite a bit of the current API can probably be replaced with a somewhat simpler functional approach.
For example, the awaitAll($promises) method (currently 50+ lines) could be expressed like this:
function awaitAll(array $promises, LoopInterface $loop)
{
    return awaitOne(Promise\all($promises), $loop);
}

(Though this skips cancellation for now, see also #4)

To expand on the previous comment, we might also consider if we should reduce our API to a single await() method can leave the remaining functional to the user instead.
await($promise, $loop);

It's sufficiently easy to call the following functions yourself:
awaitAll():
await(Promise\all($promises), $loop);

awaitRace() (see also #6 for some background):
await(Promise\race($promises), $loop);

wait() (see also https://github.com/clue/php-promise-timeout):
await(Timeout\resolve($time, $loop), $loop);

Also, combined with https://github.com/clue/php-promise-timeout, it's pretty easy to add timeout support to any of the above (see also #2):
await(Timeout\await($promise, $time, $loop), $loop);

Arguably, the $loop argument makes this a bit hard to read. Once "singularity" landed (see also first comment), the resulting call might look like this:
await(Timeout\await($promise, $time));

Similarly, this also applies to slightly more complicated combinations:
await(Timeout\await(Promise\all($promises), $time));

repo: clue/php-block-react

license: mit

license_source: bigquery


text:
Добавил Travis-CI

Теперь есть и лого на главной и каждая сборка сопровождается запуском тестов.

:thumbsup:

Владимир, у вас есть пуш-доступ к этому репозиторию - прямо и мерджите все сюда.

Окей

authors: spacediver vslutov

repo: cmc-python/modelmachine

license: WTFPL


text:
[kokkos] Go back to a dynamic library per package by disabling relocatable device code

A dynamic library per package fits better into CMSSW, so let's see how far we get without relocatable device code.

FYI @fwyzard in case you'd be interested.

repo: cms-patatrack/pixeltrack-standalone


text:
Fix meow_bench crash on Mac

posix_memalign needs alignment to be multiple of pointer size, ref #34

Thanks Aras!
- Casey

authors: aras-p cmuratori

license: Zlib


text:
[bn] adding tags in ebpf.md

Adding tags in ebpf.md
Checklist before opening this PR (put x in the checkboxes)

[ x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x ] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

If you follow this one, you can see the previous version was right: https://github.com/cncf/glossary/blob/main/content/en/ebpf.md

If you follow this one, you can see the previous version was right: https://github.com/cncf/glossary/blob/main/content/en/ebpf.md
@asem-hamid

networking and security are missing in main content . already fixed in #2868

repo: cncf/glossary


text:
[ja] Localize Security Chaos Engineering into Japanese

Describe your changes
Localize https://glossary.cncf.io/security-chaos-engineering/ into Japanese
Related issue number or link (ex: resolves #issue-number)
Resoleves #2833
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Target preview:

https://deploy-preview-3035--cncfglossary.netlify.app/ja/security-chaos-engineering/

authors: Okabe-Junya katzchang


text:
Change layout to 16:9

Signed-off-by: Jordi Noguera jordinoguera83+github@gmail.com

Deploy preview for landscape ready!
Built with commit 30b0d9ce635003693048cee11cb46d4d37cf7a11
https://deploy-preview-1449--landscape.netlify.com

LGTM

The right side of https://jordi-change-layout--landscape.netlify.com/images/landscape.png is cropped off. Please fix than LGTM again.

Fixed landscape.png to fit 16:9
LGTM

authors: CNCF-Bot dankohn jordinl83

repo: cncf/landscape


text:
Update UCloud logo

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape ready!
Built with commit 6706f25f381d6303f02ae86ec1da59e133ef21c0
https://deploy-preview-1450--landscape.netlify.com

LGTM

authors: CNCF-Bot taylorwaggoner


text:
Analyze contributor survey results

Attached is the summary of the survey results so that we can think of any further queries we would like to run against the results. The summary results does not include free-form responses.

The TAG had the survey results, but nobody showed interest in analysis of the survey data. Until someone shows interest, there's no point of doing another survey.
Closing this one as the data is outdated already.

authors: aliok carolynvs

repo: cncf/tag-contributor-strategy


text:
cnpm卡在install.js的下载二进制文件过程中

最近偶然发现有执行cnpm安装某些包时一直卡在build fresh packages阶段。经过源码分析，调用栈跟踪，发现是卡在post scripte执行过程中的二进制文件下载。比如安装optipng，执行post script阶段，会去https://cdn.npmmirror.com/binaries/optipng-bin/v7.0.1/vendor/macos/optipng
下载optipng这个文件，调用栈显示卡在获取过程中一直没有进展，但是手动下载是正常的。下图1是执行卡住的地方，下图2是正常执行的调用栈和卡住的调用栈对比。（另外发现一个有意思的现象，卡巴斯基扫描yarn cache文件会后会很容易复现此问题，并且卡住后退出卡巴斯基会立即导致卡住的node进程退出）
环境：mac os 10.15.7 (19H15)
cnpm -v
cnpm@8.1.0 (/usr/local/lib/node_modules/cnpm/lib/parse_argv.js)
npm@6.14.17 (/usr/local/lib/node_modules/cnpm/node_modules/npm/lib/npm.js)
node@18.2.0 (/usr/local/Cellar/node/18.2.0/bin/node)
npminstall@6.2.0 (/usr/local/lib/node_modules/cnpm/node_modules/npminstall/lib/index.js)
prefix=/usr/local/Cellar/node/18.2.0
darwin x64 19.6.0
registry=https://registry.npmmirror.com

卡巴斯基扫描yarn cache文件会后会很容易复现此问题

没太看懂这块， cnpm 没用 yarn cache 的。
关掉杀毒软件还能复现么？

卡巴斯基扫描yarn cache文件会后会很容易复现此问题

没太看懂这块， cnpm 没用 yarn cache 的。
关掉杀毒软件还能复现么？

只是说：用卡巴斯基扫描yarn cache 之后，会很容易复现此问题，并不知道两者之间是否有关联还是巧合。 牵扯上卡巴斯基主要是注意到，卡住的时候退出卡巴斯基，马上就会导致卡住的node进程退出（也是不知道两者是否有关联）。
关掉卡巴斯基后，还没有复现过。
怀疑1（只是猜测）: cnpmmirror有某种网络限制策略，（将下载地址换成自己建的服务器，同样有卡住的情况）；
怀疑2：可能公司网络有某种策略；
应对方案：
1）可以通过--ignore-scripts 参数跳过 post install script阶段；但是不知道有什么影响；
2）--network-timeout设置超时时间，超时失败会执行本地源码编译；  但是超时参数--network-timeout并没有对 post install script阶段生效；
下一步动作：
1）抓包观察网络方面是否有什么线索；
2）找Linux环境，尝试复现，并分析调用栈；
或者大佬有更好的建议，帮忙分析一下（如果卡住了，需要怎么分析卡住的位置和原因）

结贴：
1）本帖主要目的是找到一种方法能看到yarn或者cnpm在安装时一直没有进度是卡在哪里了；——可以通过strace调用栈看到系统调用来确认是否仍在执行以及执行的类型（下载还是编译）；
2）本帖遇到的卡住分两个原因：a.本地环境卡住，高度怀疑是杀软做了拦截，已经知会it做了白名单；b.CI 环境卡住，有些模块的post script阶段需要国外网下载一些文件导致（增加代理或者确认下载地址是否可以用环境变量来设置，或者用淘宝源（对硬编码的地址做了优化，改成了淘宝源））
3）遗留问题：准备搭建一套环境来识别模块安装时需要访问的所有地址以确认是否存在网络问题；

第二点参考 https://github.com/cnpm/cnpmcore/pull/239/files

第二点参考 https://github.com/cnpm/cnpmcore/pull/239/files ，比较常用的大包可以类似提 PR 给 npmmirror 做镜像

ok, 3Q

repo: cnpm/cnpmcore


text:
feat: add @wuxh scope & allow  refined-antd-changelog

添加 @wuxh 组织范围包，作为组织管理员，我会严格遵循社区规则。ref: https://www.npmjs.com/~wxh16144

添加 refined-antd-changelog 以使得油猴脚本正常工作

thx~

Summary by CodeRabbit

New Features

Introduced a new scope @wuxh for enhanced package management.
Added the refined-antd-changelog package to support changelog generation.

这是开库以来最优秀的 pr！

repo: cnpm/unpkg-white-list


text:
Remove the dependency on SHOW statements. Use info_schema and pg_catalog more.

Needed to fix https://github.com/cockroachdb/cockroach/issues/26983
and to complete https://github.com/cockroachdb/cockroach/pull/27098
Informs https://github.com/cockroachdb/cockroach/issues/26993
Fixes #57.
Fixes #50.

It took me a while but I succeeded: the code is now simultaneously compatible with CockroachDB 1.1, 2.0 and 2.1. (I tested.)
PTAL.
I am bumping the version because of the significant change in the versioning approach.

I bumped the date and informed justin. Merging. Thanks for all the feedback!

repo: cockroachdb/cockroachdb-python


text:
Proposal for a new sidebar organisation for sql docs.

This PR forks off #1008.
This is a proposal to initiate a discussion about the structure of the sidebar.
The motivation from this stems from the observation that the sidebar is hard to navigate in the "develop" section and fails to identify the hierarchical nature of how developers approach documentation about a new programming environment.
The proposal approaches this by defining the following top sections:

Writing SQL queries (About the SQL language as supported by CockroachDB)
Using transactions (About how the SQL queries are handled server-side)
Data definition (About how to organize data)
Misc: Security ("Privileges" in proposal) and Client Sessions (not yet written, but will need to be)

It further separates the language features that belong to each of these categories. The classification is "clean" in that it partitions the entire set of SQL statements along these lines without overlap (with one exception, "CREATE TABLE AS" which is really "CREATE TABLE" combined with "INSERT" and is a non-standard extension).

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5184f496fce1ef86c96604f846455366f2f11dc4/

Note that regardless of what you think the sidebar I would recommend adopting this structuring of sql-statements.md nonetheless, as it makes its reading much more pleasant.

Three levels of sidebar gets visually unpleasant. I like the idea of this change, but kind of wish we would remove the sidebar completely and adopt a new method of navigation, and then do changes like this.

What I'd like is a proper book structure with parts, chapters and sections. Ultimately we ought to produce a single PDF with the whole. I am not too attached to the sidebar, but until we have a proper table of contents and navigation items (like prev/next section buttons) we need to do something about it.

I agree with @mjibson, three levels of sidebar navigation is too much for these common pages, especially when the sidebar collapses again when you navigate. I like the proposed grouping of SQL statements but I'd prefer to combine this with lifting the whole "SQL Statements" subtree up to the top level (I still have to stop and think for a moment to locate this section under the "Develop" heading).

As mentioned to @knz in another thread, I think the direction of these changes are great, but I'm a bit concerned about tackling too much of this this quarter, for a few reason:

To properly rethink the architecture, and possibly move away from a sidebar, we need Design (@kuanluo) to help us plan and test for that, ideally with actual users, so as to get away from our own biases. But site re-architecting isn't in scope for Design or Docs for Q1. Likely Q2.
@mjibson is planning to migrate docs from Jekyll to Hugo along with the Wordpress portion of the site. That'll free us up to put in place an actual site/url structure (right now, everything's flat).

Here's what I propose:

We come up with a short-term fix to raise SQL to the top level (building off of the changes @knz has here).
In migrating to Hugo, Matt keeps the docs hierarchy flat for now. If we build in a url hierarchy with the move and then change the hierarchy again with our re-architecting, that's a lot of headache for us and users (lots of redirects).
We start planning a larger overhaul of the site structure, to be continued, tested, and implemented in Q2.

Thoughts all, especially @kuanluo and @mjibson, since this alters the migration plan for this quarter?

That sounds ok to me. I don't think a jekyll -> hugo conversion will take long. There are programs that automate most of it already.

My thoughts echo @jseldess; to do this effectively, we need to engage design and make data-driven decisions about restructuring the site and its navigation. Before signing up for any major restructure, I need data validating its efficacy vs. other possibilities. This specific restructure, though, is definitely worth including in those tests.

Closing this for now. We should reference the ideas and conversation here once we prioritize reworking the docs info architecture.

authors: bdarnell cockroach-teamcity jseldess knz mjibson sploiselle

repo: cockroachdb/docs


text:
Update search behavior

[description]:

update search script to search against numeric docs version instead of version tag
update broken styling of sidebar bottom border

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f85cc44057972c773a2858ba0608c567799722ad/

authors: cockroach-teamcity j-low


text:
\demo_node docs

Fixes #6070.
Added syntax table and examples for cockroach demo cluster SQL \demo_node commands.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2d64ad30e774359f38827508f196e322f3a16585/
Edited pages:

v20.1/cockroach-demo.md

v20.1/cockroach-demo.md, line 257 at r1 (raw file):

### Shut down and restart nodes

If you start a demo cluster with multiple nodes, you can use the [`\demo_node`](cockroach-demo.html#shutting-down-and-restarting-nodes) to shut down and restart individual nodes in the demo cluster.

can you add a note this is currently experimental?
(we haven't documented it on the cli yet for this reason :))

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b104e040f8e48f7c0fda592a6e1a9f6c91401843/
Edited pages:

v20.1/cockroach-demo.md

authors: cockroach-teamcity ericharmeling otan


text:
roachprod create exploded in my face with no warning (also, its handling of ssh/RSA is bogus)

kena@kenabook ....com/cockroachdb/cockroach % roachprod create knz-hibernate -n 1

Creating cluster knz-hibernate with 1 nodes
OK
knz-hibernate: [gce] 12h7m40s remaining
  knz-hibernate-0001    knz-hibernate-0001.us-east1-b.cockroach-ephemeral       10.142.0.2      35.237.126.26
Syncing...
knz-hibernate: waiting for nodes to start 1/1
generating ssh key 1/1
distributing ssh key 0/0
retrieving hosts 1/1
scanning hosts 1/1
0: exit status 1
~
for i in {1..20}; do
  ssh-keyscan -T 60 -t rsa Warning: Identity file /var/home/kena/.ssh/id_rsa not accessible: No such file or directory.
10.142.0.2 35.237.126.26 > .ssh/known_hosts.tmp
  if [ "$(cat .ssh/known_hosts.tmp | wc -l)" -eq "2" ]; then
    cat .ssh/known_hosts.tmp >> .ssh/known_hosts
    rm -f .ssh/known_hosts.tmp
    exit 0
  fi
  sleep 1
done
exit 1

Warning: Identity file /var/home/kena/.ssh/id_rsa not accessible: No such file or directory.
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo not: Name or service not known                                                                                [210/9018]
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname                                                                                [112/9018]
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname                                                                                [12/9018]
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found
getaddrinfo Warning:: Name or service not known
getaddrinfo Identity: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo /var/home/kena/.ssh/id_rsa: Name or service not known
getaddrinfo not: Name or service not known
getaddrinfo accessible:: Name or service not known
getaddrinfo No: No address associated with hostname
getaddrinfo such: Name or service not known
getaddrinfo file: Name or service not known
getaddrinfo or: Name or service not known
getaddrinfo directory.: No address associated with hostname
bash: line 3: 10.142.0.2: command not found

github.com/cockroachdb/roachprod/install.(*SyncedCluster).SetupSSH.func4
        /home/spill/go/src/github.com/cockroachdb/roachprod/install/cluster_synced.go:546
github.com/cockroachdb/roachprod/install.(*SyncedCluster).Parallel.func1.1
        /home/spill/go/src/github.com/cockroachdb/roachprod/install/cluster_synced.go:1115
runtime.goexit
        /usr/lib/go-1.10/src/runtime/asm_amd64.s:2361:
2018/10/20 16:52:50 command failed

Several problems here:

roachprod is trying to use the words an error message as a list of IP addresses/hostnames. This smells like improperly mixing stderr with stdout.

the thing is inflexible in requiring (and reading) a private RSA key in the filesystem.

roachprod has no business reading private keys. It should stick to the public keys presented by the SSH key agent (ssh-add -L?).
this design is unsound in the presence of secure RSA tokens where the private part of the key is off-system (but the key identity properly presented by the key agent)
there is no reason to mandate RSA. roachprod should accept to take anything that comes up from the key agent, including ECDSA keys.

cc @benesch @petermattis

I totally agree that our handling of SSH keys needs some love, but using just the keys available in the SSH agent had its own problems. We have a number of less technical PMs using roachprod now and properly configuring an SSH agent was proving to be a burden. Plus GCE workers and TeamCity tend not to have SSH agents configured.
The goal was to make it so that roachprod tries all the keys in your agent as well as ~/.ssh/id_rsa and ~/.ssh/google_compute_engine. The latter two really increase the convenience for someone who wants roachprod to "just work". I didn't realize that SSH would throw such a fit if you specified -i KEYFILE without KEYFILE actually existing. I think the solution here might be as simple as not specifying -i ~/.ssh/id_rsa unless ~/.ssh/id_rsa actually exists.

Explosion is an apt description. Either Nikhil or I will patch something up. @knz PRs are welcome if you feel compelled to do something more significant here.

The inconvenience was pretty minor -- the explosion was verbose enough to tell me what to do.
So despite the amount of bytes involved I'd say this is pretty low priority.

authors: benesch knz petermattis

repo: cockroachdb/roachprod


text:
fix: execute hooks directly instead of through sh

On Windows my hooks weren't being ran. I noticed it was because sh was being used. I instead made the hook accept a list where the assumption is the first element in the list is the program and the rest are the arguments.
It works well for me, but the tests are failing because they assume everything is passed through sh -c. If this change is acceptable I can attempt to update the rest of the tests.
This is technically a breaking change, but what do you think?

Hey @zaucy, hooks are run in a subshell to be able to expand environment variables. So I can't merge this as it is.
If you have a unix environment available (maybe through wsl2) you can test it with the following:
    #[test]
    fn run_hook() {
        let hook = Hook::from_str("sh -c 'echo $HOME'").unwrap();
        hook.run(None).unwrap(); // print "/home/okno"
    }

    #[test]
    fn run_hook_without_subshell() {
        let hook = Hook::from_str("echo $HOME").unwrap();
        hook.run(None).unwrap(); // print "$HOME"
    }

If it's not working on windows we could fix it with some conditional compilation attributes. Or ideally, provide a way to specify the subshell when writing hooks, including powershell.

superseded by #304

authors: oknozor zaucy

repo: cocogitto/cocogitto


text:
[Plugin-x] FacebookAgent api function should treat non-text type differently

Otherwise, it will lead to crash on native platforms
Originally reported : http://discuss.cocos2d-x.org/t/my-dilemma-with-loading-online-image-from-facebook/19026/3

Hi,
Is it easy to fix manually?

authors: pandamicro shirkan


text:
Dumps

We identified with Romain 2 bugs in dumps:

People that are admins but NOT competition owners cannot create dumps
Newly created dumps (e.g. today june 8, 2020) obtain the "wrong" date.

 in this example the date 2019-11-27 should be 2020-06-08

There is no possibility to create a dump of competition that includes the task as a zip inside the competition bundle.

Merged.

authors: Didayolo madclam

repo: codalab/codabench


text:
counter_for loop block: Add new i18n string to all locales

A one-off script was written to add a new i18n string used in the counter_for loop block to all locales.  This ensures that the counter_for loop block is not broken when viewed in a non-US locale.  Follow up work is to eliminate this manual process.

Some questions from someone who has been out of the loop:

Will these strings be added to the correct source files so that they will persist between i18n syncs?
I think these strings should be added to the apps i18n files not to the files in the i18n directory?
Can you share the code you used to do this? I would love to be able to look back on these PRs and see what to do to solve these issues in the future!

@bethanyaconnor
Here are responses to your questions:

Yes, these strings will be added to the correct source files.
When I run this script it will add the strings to apps i18n
Sure, I will update PR to include script.

authors: bethanyaconnor nkiruka

repo: code-dot-org/blockly


text:
Feature/roles

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Fixed issues (PRs without this will not be accepted)

Fixes #280

Changes

Create Role Module (resolver, services, guards, decorator, entity)
Integrate Role Module with User module ( add role to user, remove role to user, create user with default role)
Add seeders in seeders.services

PR Checklist

[ ] Does your PR contain relevant tests that pass?
[ X] Is your code linted? yarn lint or yarn lint:fix
[ X] Does your code build correctly in ALL projects? yarn build

Hey @RosoPenaranda how's it going with this one?

I'm going to take over this one tonight, as it's over a month old

repo: code-mentoring/learn

license: Unlicense


text:
Can't run new logging component witout Elasticsearch.

Does CRC1.18 (or OCP 4.6.1) support logging component without Elasticsearch?
I set up ClusterLogging instance and ClusterLogForwarder, only to output syslog to external server.
So all I want is fluentd daemonset. like this yaml.
ClusterLogging instance
apiVersion: "logging.openshift.io/v1"
kind: "ClusterLogging"
metadata:
  name: "instance"
  namespace: "openshift-logging"
spec:
  managementState: "Managed"
  collection:
    logs:
      type: "fluentd"
      fluentd:
        resources: null

ClusterLogForwarder instance
apiVersion: logging.openshift.io/v1
kind: ClusterLogForwarder
metadata:
  name: instance
  namespace: openshift-logging
spec:
  outputs:
   - name: rsyslog-east
     type: syslog
     syslog:
       facility: local0
       rfc: RFC3164
       payloadKey: message
       severity: informational
     url: 'udp://10.146.0.6:514'
  pipelines:
   - name: syslog-east
     inputRefs:
     - audit
     - infrastructure
     - application
     outputRefs:
     - rsyslog-east
     labels:
       syslog: east

But instance output this status
clusterlogging instance status
$ oc get clusterlogging instance -o yaml
apiVersion: logging.openshift.io/v1
kind: ClusterLogging
metadata:
  creationTimestamp: "2020-11-17T05:56:28Z"
  generation: 1
  managedFields:
  - apiVersion: logging.openshift.io/v1
    fieldsType: FieldsV1
    fieldsV1:
      f:spec:
        .: {}
        f:collection:
          .: {}
          f:logs:
            .: {}
            f:fluentd:
              .: {}
              f:resources: {}
            f:type: {}
        f:managementState: {}
    manager: Mozilla
    operation: Update
    time: "2020-11-17T05:56:28Z"
  - apiVersion: logging.openshift.io/v1
    fieldsType: FieldsV1
    fieldsV1:
      f:status:
        .: {}
        f:clusterConditions: {}
        f:collection:
          .: {}
          f:logs:
            .: {}
            f:fluentdStatus: {}
        f:curation: {}
        f:logStore: {}
        f:visualization: {}
    manager: cluster-logging-operator
    operation: Update
    time: "2020-11-17T05:57:47Z"
  name: instance
  namespace: openshift-logging
  resourceVersion: "262714"
  selfLink: /apis/logging.openshift.io/v1/namespaces/openshift-logging/clusterloggings/instance
  uid: 6adc8162-b525-4c7a-9683-87186d4596e3
spec:
  collection:
    logs:
      fluentd:
        resources: null
      type: fluentd
  managementState: Managed
status:
  clusterConditions:
  - lastTransitionTime: "2020-11-17T05:57:47Z"
    status: "False"
    type: CollectorDeadEnd
  collection:
    logs:
      fluentdStatus: {}
  curation: {}
  logStore: {}
  visualization: {}

clusterConditions status is False, and type is CollectorDeadEnd.
I believe ClusterLogForwarder has no problem
$ oc get ClusterLogForwarder instance -o yaml
apiVersion: logging.openshift.io/v1
kind: ClusterLogForwarder
metadata:
  creationTimestamp: "2020-11-17T05:57:47Z"
  generation: 1
  managedFields:
  - apiVersion: logging.openshift.io/v1
    fieldsType: FieldsV1
    fieldsV1:
      f:spec:
        .: {}
        f:outputs: {}
        f:pipelines: {}
    manager: Mozilla
    operation: Update
    time: "2020-11-17T05:57:47Z"
  - apiVersion: logging.openshift.io/v1
    fieldsType: FieldsV1
    fieldsV1:
      f:status:
        .: {}
        f:conditions: {}
        f:inputs:
          .: {}
          f:application: {}
          f:audit: {}
          f:infrastructure: {}
        f:outputs:
          .: {}
          f:rsyslog-east: {}
        f:pipelines:
          .: {}
          f:syslog-east: {}
    manager: cluster-logging-operator
    operation: Update
    time: "2020-11-17T06:20:50Z"
  name: instance
  namespace: openshift-logging
  resourceVersion: "271831"
  selfLink: /apis/logging.openshift.io/v1/namespaces/openshift-logging/clusterlogforwarders/instance
  uid: 201156a5-94c7-4701-92e6-20d18b390c79
spec:
  outputs:
  - name: rsyslog-east
    syslog:
      facility: local0
      payloadKey: message
      rfc: RFC3164
      severity: informational
    type: syslog
    url: udp://10.146.0.6:514
  pipelines:
  - inputRefs:
    - audit
    - infrastructure
    - application
    labels:
      syslog: east
    name: syslog-east
    outputRefs:
    - rsyslog-east
status:
  conditions:
  - lastTransitionTime: "2020-11-17T06:20:50Z"
    status: "True"
    type: Ready
  inputs:
    application:
    - lastTransitionTime: "2020-11-17T06:20:50Z"
      status: "True"
      type: Ready
    audit:
    - lastTransitionTime: "2020-11-17T06:20:50Z"
      status: "True"
      type: Ready
    infrastructure:
    - lastTransitionTime: "2020-11-17T06:20:50Z"
      status: "True"
      type: Ready
  outputs:
    rsyslog-east:
    - lastTransitionTime: "2020-11-17T06:20:50Z"
      status: "True"
      type: Ready
  pipelines:
    syslog-east:
    - lastTransitionTime: "2020-11-17T06:20:50Z"
      status: "True"
      type: Ready

CRC is one node, so we can't run Elasticsearch.  nodeCount: 3 is not allowed in ClusterLogging instance.

founnd this.
https://bugzilla.redhat.com/show_bug.cgi?id=1891738

repo: code-ready/crc


text:
Adds tracks sorting for better navigation

PR Fixes:

Adds a dropdown just below the category filter bar/dropdown.
Sorting is based on alphabetically ascending & descending, newest first, oldest first orders.
Improves navigation.
Fix typos in function name from fitlreTracks to filterTracks removed an "e"

Screenshots:
List closed:

List opened:

Resolves #385 #178 #58
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@hkirat could you please review this PR :)

/bounty $20


text:
Add node engine version in package.json

This will give developers a warning if they are using incorrect versions of node when installing the project.
I believe this project uses Node 18 or higher.
See the following for how to use it: https://docs.npmjs.com/cli/v10/configuring-npm/package-json#engines

Version >=20 was defined in this commit https://github.com/code4sac/opensac.org/pull/50/commits/204d438fab0a30731dfb3bbd72c2924c67a38bc5.
The project minimum Node version is 18.17 which is required by Next.js 14.

authors: feydan natebass


text:
🛑 CodeAdjust Cloud is down

In 4bb027f, CodeAdjust Cloud (https://cloud.codeadjust.com/robots.txt?$GITHUB_RUN_ID) was down:

HTTP code: 521
Response time: 367 ms

Resolved: CodeAdjust Cloud is back up in 6fce5f3 after 6 minutes.

repo: codeadjust/services-status


text:
[Feature Request] add an condition to check staleness of an element

Hi @asolntsev,
Similarly like existing conditions like $("element").shouldBe(visible) can you add a condition which checks staleness of an element.
Like this:
$(element).shouldBe(stale) or better. Let me know your opinion on it.
Thanks,
Anil

@anilreddy Can you comment why it's useful? What is the use case?
(Currently I don't imagine why I could need it. It seems that real end-user also don't see "stolen" elements, so test also should not.)

sorry its my test issue closing it.

@anilreddy Still waiting for your answer...

@asolntsev I was on vacation. Currently I didn't find good example that we have in our application. I will open it again when I find one. Am closing it for now.

authors: anilreddy asolntsev

repo: codeborne/selenide


text:
String escape

Hi there.
I would like to know if there is a function that escapes simple quotes from queries and if it does exists, how to use it, because I have not found it (yet).
If this does not exists, it would be great if you could implement it!
Thanks. #

http://stackoverflow.com/questions/603572/how-to-properly-escape-a-single-quote-for-a-sqlite-database

authors: Andromelus robertohuertasm


text:
Visual/UX Style Guide

When @bdickason was making some awesome mockups, he asked if their was a UX mockup style guide for CodeCombat. We don't have one yet, and it would be pretty good to have in the future.

Now at http://codecombat.com/admin/design-elements!

authors: nwinter schmatz

repo: codecombat/codecombat


text:
Autocomplete that is triggered while playing,doesn't disappear on pressing the back button.

After the autocomplete is suggested,when the back button on browser is pressed, the screen is loaded along with the autocomplete which doesn't go away until you choose one of those options.

I'm having trouble re-producing the error. What browser and system are you on?

I used firefox on ubuntu 14.04

I have the same issue with firefox 35.0.1 on ubuntu 14.04

Not seeing it; anyone still having issues with this?

authors: matifol navya-nizamkari nwinter rmarren1


text:
How do I be a admin on CodeCombat?

I wanted to be a admin on CodeCombat. Is there a way to be a admin on CodeCombat?

Admins are only for codecombat staff. If you think you want to be a staff member go here: https://codecombat.com/about#careers

Thanks @Chaboi45 !


text:
Inline scripts should be externalized

In chat/room page, some javascript from plugins are inline scripts.
It's bad for performance.
So they should be externalized.

Coverage increased (+0.01%) to 89.8% when pulling 7520125158f3f96377ee178c4238826558c805f2 on mallowlabs:id/209 into 16e47335240f0318b6db2d92be81adc5c1ba1a46 on codefirst:master.

Thanks :rabbit2:

authors: coveralls mallowlabs suer

repo: codefirst/AsakusaSatellite


text:
Vartul Bahuguna added

Profile added

Welcome to Code for Cause :tada:

authors: GangaChatrvedi VartulBahuguna

repo: codeforcauseorg/pledge


text:
Implement HTTPS for authentication

Sending plaintext passwords over the internet is a massive security hole.  Any site that requires authentication should implement HTTPS for basic channel security.
https://aws.amazon.com/certificate-manager/ should be able to do it.
This can probably also be done using LetsEncrypt.

In addition to the SSL certificate, you also need to force redirects to https.
Apache instructions / snippet:
https://www.namecheap.com/support/knowledgebase/article.aspx/9821/38/apache-redirect-to-https
nginx instructions / snippet:
https://serversforhackers.com/c/redirect-http-to-https-nginx

authors: aliu-vmware sunnymui

repo: codeforsanjose/disaster-response-sj


text:
需要获取那些呀

获取

刷新一下网页然后找一个网页请求（任意cookie包含有STOKEN字段的）的全部cookie扔进去解析就行。


text:
Error Send Email with CodeIgniter v4.1.3

Direction
We use github issues to track bugs, not for support.
If you have a support question, or a feature request, raise these as threads on our
forum.
Describe the bug
I get an error when sending an email with config protocol smtp and codeigniter version 4.1.3
Message Error:
Email: sendWithSmtp throwed fsockopen(): php_network_getaddresses: getaddrinfo failed: No such host is known.
But with same email config I successfully sending an email with codeigniter version 4.0.4
CodeIgniter 4 version
v4.1.3
Affected module(s)
Not sure but looks like is /system/Email/Email.php
Expected behavior, and steps to reproduce if appropriate
Should successfully sending email with codeigniter version 4.1.3
Context

OS: Windows 10
Web server Apace
PHP version 7.4.18

This is not a bug from the framework but an issue in your connection (DNS, etc.). Check this forum link for debugging steps.
https://forum.codeigniter.com/thread-16736.html

oke thanks @paulbalandan

authors: ferdyciputra paulbalandan


text:
Original email port

This is the original Email port from CodeIgniter 3.
It is inelegant and tested by assumption only (it worked it CI3, right?).
It is, however, available "now", as a "better-than-nothing" provision for CodeIgniter 4's initial release.
I will continue to work on a more elegant and tested Email package, targeting 4.1.

Bit of a hit on code coverage :(

Closed? Could this mean... Email 4.0?

I'm cool with this if it was closed by mistake. Feel free to merge if that's the case.

Dropped by mistake, sorry.

authors: MGatner jim-parry lonnieezell


text:
fix: is_cli() returns true when $_SERVER['HTTP_USER_AGENT'] is missing

Description
Fixes #5336
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[] User guide updated
[x] Conforms to style guide

These were changes pretty recently by @paulbalandan to address other issues - he should take a look.

Thanks.
#4699 is cgi-fcgi.
#4739 does not show why. What's the issue to solve?
#4843 is fpm-fcgi.
#4848 is fpm-fcgi.

Obviously, accepting this fix would fail cronjobs in cgi fixed previously.

I agree with it.
It seems there are users who do not use php-cli with cron.

@paulbalandan I added conditions.
I removed HTTP_USER_AGENT and argv conditions, because #5336 has argv and no HTTP_USER_AGENT with Apache environment.

#4739 was raised by @najdanovicivan in the Slack channel. I cannot remember the issue then, but adding the argv checks fixes his issue.

Searched slack, and I found Ivan said the following causes the issue.
if (! isset($_SERVER['REMOTE_ADDR'], $_SERVER['HTTP_USER_AGENT']))
{
   return true;
}

And said $_SERVER[‘argv’] is not set.
The following is the fixed code at that time:
		if (PHP_SAPI === 'cli')
		{
			return true;
		}

		if (defined('STDIN'))
		{
			return true;
		}

		if (stristr(PHP_SAPI, 'cgi') && getenv('TERM'))
		{
			return true;
		}

		if (! isset($_SERVER['REMOTE_ADDR'], $_SERVER['HTTP_USER_AGENT']) && isset($_SERVER['argv']) && count($_SERVER['argv']) > 0)
		{
			return true;
		}

		// if source of request is from CLI, the `$_SERVER` array will not populate this key
		return ! isset($_SERVER['REQUEST_METHOD']);

@najdanovicivan Do you have any problem with this PR?

@najdanovicivan Do you have any problem with this PR?

All good here. I had issues with some web requests being treated as CLI due to missing HTTP_USER_AGENT

💪

authors: MGatner kenjis najdanovicivan paulbalandan


text:
Workaround to Clang crashes with nested tuples

Hi! Following issue #26, I have found a workaround that seems to solve clang problems.
The proposed change, however, turns off the use of concepts in the tuplet::tuple::assign function.
From what I have seen, in this case, concepts are used for better error messages and not to choose the correct overload, so everything should work just fine (with worse error messages though).
Here is a godbolt testing compilation from clang 10 to 15.

Messing around with the requirements syntax, I have found a way to constrain the types that also work with Clang.
It seems that require clauses are friendlier to clang than concepts in the template parameters declaration.

repo: codeinred/tuplet


text:
Navigation Bar previous active link is still active

When we use the navigation bar to move from one page to another, on using the back button of the browser, the previously active page still remains active. This may confuse the user on exactly which page he/she currently is. I checked on Google Chrome as well as Microsoft Edge, both the browsers show the same error.
Steps to reproduce the behavior:

Travel some pages using navigation bar.
Then using the back button of browser go to previous pages.
You shall see that the previously active tab is still active even though we are not on that page.

Expected behavior:-
Only the current page's corresponding link/tab should be active. This will not confuse the user of current page's title.
Screenshots
https://user-images.githubusercontent.com/71634100/162723279-126cd380-fa5f-4809-91cc-ca74d9c77289.mp4
Desktop

OS: Windows 10
Browser : Google Chrome, Microsoft Edge

Hello, Greetings. If this is valid ,may I work on it?

I think if you change the effect then will work. I can do this .

Oh, thanks for recommending., but I actually found the solution for this.

Good catch, thankyou @CoderS2020 , but please wait for a little before a maintainer assigns you an issue.

@CoderS2020 no worries!

authors: CoderS2020 padmajabhol rahoolonly


text:
pypi over SSL

@codekoala We installed the pypi docker version on our mesos environment. We enabled SSL for accessing pypi.
When we hit https://pypicloud.staging.net/ it is giving a blank page. But a curl returns 200 OK status.
https://pypicloud.staging.net/simple/ gives the list.
Any thoughts on how to have pypi working with SSL? Please suggest.

Whoa, I apologize for the crazy late response! Lots of messages fall through the cracks far too often.
If you still need help with this, a simple option comes to mind: terminate SSL with a reverse proxy such as nginx and proxy http requests through to pypi-server. You could either run the reverse proxy separately or build your own Docker image based on this one and simply run both the proxy and pypi-server within the same image.
Hope that helps. Sorry again for the delay.

repo: codekoala/docker-pypi


text:
Does anyone know a javascript alternative to this library?

My search so far tells me that the ML/Data science tools built in python are light years ahead of any other language (as expected). (Though I'd still like to maintain single language stack if possible)
Does anybody know a Node/JS alternative of this library?

@AdityaAnand1 newspaperjs library looks good. but i didnt try it any.

@AdityaAnand1 this works perfectly: https://github.com/postlight/mercury-parser

authors: AdityaAnand1 GabrielePicco ilkerceng

repo: codelucas/newspaper


text:
막대그래프에 재산 데이터가 표시되지 않는 경우

아래 경우처럼 특정 연도들(주로 2010년 이후의 연도)의 총액이 표시되지 않는 경우가 꽤 있습니다.
박근혜는 동명이인이 없는 경우인데, 소계 데이터에 동명이인을 잘못 처리해서 생기는 문제는 아닌 것 같네요.

👍

repo: codenamu/official-assets-explorer


text:
Change Marker icon

I used to define new types of marker with Leaflet.Draw, each one with different options, that feature was very useful and scalable, It will be great to implement a similiar mechanism in this library.
Actually I'm searching a way (elegant) to set a new marker icon for markers drawn by user.
Any suggestion?

mhh I understand what you want. This would be possible with custom options (including CSS classes and icon images).
Can you post a screenshot about the dialog that leaflet.draw uses so you can pass options during draw (without coding)?

@codeofsumit I didn't get what you asked me. I don't use Leaflet.Draw anymore, because it doesn't work well for touch event instead this library works very well on mobile device.
I just say that was good idea to extends Marker class to realize custom Marker instance.
Now I just want to apply a new icon for a marker, I tried to do this, but doesn't work:
var options = { icon: 'img/newicon.png' } map.pm.enableDraw('Marker', options);

ahh ok sorry I misunderstood. I'll implement it as soon as I find the time (hopefully this week) 👍  thanks for the suggestion!

you can soon do this via
// optional options for line style during draw. These are the defaults
var options = {
    // custom marker style (only for Marker draw)
    markerStyle: {
        opacity: 0.5,
        draggable: true,
    }
};

// enable drawing mode for shape - e.g. Poly or Line
map.pm.enableDraw('Marker', options);

you can try it out with the develop branch. I'll implement one more feature before I release

Document

@codeofsumit The above solves for when you call map.pm.enableDraw('Marker', options); manually in code. But what about when a user clicks on the 'Draw Marker' button from the map?

I think you can pass this to global options now (check the docs) and it will apply also to the toolbar.
Can’t test right now I’m on the phone

authors: VCodepp codeofsumit giomarino jziggas


text:
support scoped variable via proper compile pass

Scoped-variable is now implemented via a proper compile pass. It is enabled by default. But it is not stable enough, you can toggle it off in the sidebar.
The following visual appearance is added to the editor:

function definition
(global) variable definition
function call site
variable use
unbound variables

Screenshot:

UPDATE: hide annotations by default, and add a "show annotations" setting to toggle it.

repo: codepod-io/codepod


text:
🛑 Samuel Mach is down

In dd7f769, Samuel Mach (https://samuelmach.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Samuel Mach is back up in 29d9fa4 after 2 hours, 58 minutes.


text:
uni.addInterceptor拦截request的问题

项目有可能出现需要请求2个不同的后端，请求参数结构和响应参数结构完全不一样的，这时候就需要创建2个http实例来分别对应2个后端，但现在使用了uni.addInterceptor把所有的请求都拦截了，都在一个http实例处理了
我只在以前的公司碰到过类似的情况，不止要请求后端接口，部分数据还要额外请求ERP的接口，因为请求的根路径和响应的数据结构与我们公司的后端都不一样，这个时候我一般都是再创建一个http实例，请求公司后端使用实例A，请求ERP使用实例B，2者互不干扰

对的，你这个需求很常见，是我考虑不周。我改进下，感谢~

后面讨论了下，有2个方案：
1）前端处理，多个http实例（不推荐）。http拦截器里面，baseUrl 不用自动拼接到 完整url, 通过实例http 传入。
2）服务器转发，分别考虑本地开发和线上环境。本地开发使用 vite的本地服务器，线上使用 nginx转发。

因为本需求目前只个别人遇到，所以不打算更改现在的实现方式。

authors: codercup dhanweb

repo: codercup/unibest


text:
イベント履歴収集スクリプトでプロバイダ指定を可能に

背景
毎週月曜に前週分のイベント履歴を収集しているが、facebook のみ 2018/04 以降収集できていなかった。
このような場合、対象期間とプロバイダを指定してイベント履歴を収集し直したい。
やりたいこと

イベント履歴収集スクリプトで期間だけでなくプロバイダを指定して実行可能にする
rake タスクの usage を修正する ＋ ファイル配置を見直す

Fix #378
このPRでやること

[x] タスクの引数にプロバイダを追加し、対象プロバイダのみイベント履歴を収集できるようにする
[x] usage を修正する
[x] 対象プロバイダ判定の rspec 追加する

やらなかったこと
特になし
レビューポイント

対象プロバイダの判定

困ってること
特になし

こちらも実装 & RSpec 追加、完了しました。
お手隙のときにレビューお願いします。 🙏  > @yasulab @nalabjp

facebookが実質使えなくなってしまった

これちょっとアイデア待ちですね 🤔💭
個人的には Facebook アプリに登録してもらうのはそれなりにコストがかかるので何か良い方法がないかなぁと考えていますが (できれば Doorkeeper や connpass のような一般公開情報だけに絞って取得したいと考えているのですが)、もし Facebook アプリでやるとしたら、 https://coderdojo.jp/kata#support に 【協力依頼】 というセクションを用意して、そこで Facebook アプリを承認してもらうという感じですかね。
今の所このぐらいしかアイデアがない状況です 😓.oO(もっといい方法ないかなぁ)

facebookが実質使えなくなってしまった

これちょっとアイデア待ちですね 🤔💭

はい。
まずは 2018 年の履歴を収集することを優先して、yaml 投入方式で回避している、という認識です。
そのため、インタフェースは残し、期間指定もできるようにしました。

今の所このぐらいしかアイデアがない状況です 😓.oO(もっといい方法ないかなぁ)

Facebook のイベント履歴、よい策があれば自動収集したいので Issue 立てておきます。

authors: chicaco yasulab


text:
consider linking https://www.websiteplanet.com/blog/what-is-open-source-software/

received as a tip via support email

not linking this for now

That's fair. It's not very code-oriented.

authors: EzequielBruni wikfeldt


text:
Write a Ruby program to print numbers from n to 1

Description
Write a Ruby program to print numbers from n to 1
Input  : 5
Output : 5 4 3 2 1

How to contribute

Save the solution in program/PrintNumbersFromNTo1.rb file
Add PrintNumbersFromNTo1.rb file in print-numbers-from-n-to-1 folder

!assign

repo: codinasion/program


text:
Bombs Away in C#

This is a Visual Studio 2022 solution and uses the latest language features like file-scoped namespaces.

Excellent, thank you!

repo: coding-horror/basic-computer-games


text:
Java 8 builds are broken CQEngine sun.reflect.ConstantPool

For IndexedMapPersisted and IndexedMapVolatile all test cases fail with the error
java.lang.NoClassDefFoundError: sun/reflect/ConstantPool

No longer an issue.

Wohoo JDK 9 build finally passed !
https://travis-ci.org/codingchili/chili-core/jobs/444160522/config

repo: codingchili/chili-core


text:
Do you plan to support Windows platform?

Are you planning to support windows in these projects (portable-python and runez, etc)?  If so, I suggest adding windows builds to your CI.  If not, it's sad because portable-python project is so perfect in terms of workflow.
I ask because I just opened this issue and I'm not sure that this project could have ever run successfully on Windows with this condition existing: https://github.com/codrsquad/runez/issues/5

I'd love to get around to making it work on windows, it's not a priority for me right now because I don't have windows use-cases yet.
I did a little bit of exploring, fixing a few things in runez on Windows a long time ago.
I agree, would be super cool to make it all work on Windows too.

Ok, thank you for the fast feedback.  Perhaps consider adding a statement to the readme that this doesn't currently support Windows, because I got pretty far along in a POC before hitting this and it was super surprising and disappointing.  It's on me for making assumptions because I got used to most python CLI apps working across all three major platforms by default. So, no hard feelings, just a shout out to help future consumers.

Thank you, I've added now a table showing (in the main README) which operating systems are supported, put a red cross mark for Windows for now (will change to green checkmark when it starts working there).

authors: solvingj zsimic

repo: codrsquad/portable-python


text:
Components Nav Link Does Not Navigate Correctly

When clicking on the "Components" link in the navigation menu, instead of navigating to the correct "Components" page, it redirects to https://animata.design/docs/background/animated-beam. This issue prevents users from accessing the intended "Components" page directly from the navigation link.
Steps to Reproduce:
Open the Animata site.
Click on the "Components" link in the navigation menu.
Observe that the site redirects to https://animata.design/docs/background/animated-beam instead of the "Components" page.
Expected Behavior:
Clicking on the "Components" link should navigate directly to the "Components" page.
Actual Behavior:
The link incorrectly navigates to https://animata.design/docs/background/animated-beam.
Additional Information:
Browser: [Brave]
OS: [Win11]
Potential Impact:
Users may find it difficult to access the "Components" page directly from the navigation menu, leading to a confusing user experience.

try this
https://bit.ly/4e4zLlq
select the desired compiler in the installer

@ItsLasa so can u tell me what is component page here? any link to component page??

What I saw is that I would need you to create the /components page. I noticed that your code has pages for blogs and docs.  Please clarify the requirements, and I'm willing to assist.

Or is the component page the homepage?  https://animata.design/

@webdevNYC  need to include All Components into  /components page

Sounds interesting. I would love to help with that. I have a ticket assigned to me that I'm planning to work on it tomorrow. But if not in a rush, I would love to create it.

repo: codse/animata


text:
bookmark always returns null

Hi,
so following Part 6 got me to my first error I couldn't solve yet.
In my BookmarksController.js my bookmark will always be null.
const {songID, userID} = req.query;
    const bookmark = await Bookmark.findOne({
           where: {
               SongId: songID,
               UserId: userID
           }
       });
res.send(bookmark || {});

And my bookmark.json file looks like this:
[
  {
    "SongId": 1,
    "UserId": 1
  }
]

Can you maybe have a short look on the repo or do you have any clue why it does not find anything? https://github.com/akratzel/guitar-tabs/tree/dev
Maybe a sequelize version issue?
Thanks!

i've the same issue, you solved?

For me it worked I changed
const {songID, userID} = req.query;
to
 const songId = req.body.params.songId    const userId = req.body.params.userId

Have you solved the issue? My query just take the songId but not the userId so it always gives me 500 error. Tried few things, but none worked..

authors: akratzel clirimajeti fuoridallarete negrodebobures

repo: codyseibert/tab-tracker


text:
Make _BlobReadStreamController a ClosableResource

_BlobReadStreamController depends on a database and allocates memory and should therefore integrate with resource management.

Closed by #80.

repo: cofu-app/cbl-dart


text:
What does parentId do in the code?

After reviewing and debugging the code, I'm unable to understand the purpose of parentId, since I want to add some message manually os I'm unable to use context in the class(I created for manual adding message),
for adding message manually to the chat I'm using sendMessage() method from chatManager class.
can someone explain why parentId is used for

Good question! Each message in a chat is considered a 'reply' to the one above it. The parent ID is a reference to that earlier message.
If you're wondering why this is needed:
A message can have multiple replies (multiple messages with the same parent ID) when the Regenerate or Edit features are used. In both cases, a new message object is created with the same parent ID as the original. You can think of it like a tree structure. Whenever Regenerate or Edit is used, the tree of messages branches off in new directions.
Tracking the parent IDs makes it possible to list out a single path down through the branches of that tree, for display in the UI and also passing as context to the AI.

authors: MirzaHasnat cogentapps

repo: cogentapps/chat-with-gpt


text:
OrderbookSync does not reconnect when socket connection is lost

When the socket connection is lost, OrderbookSync ends up in a stale state (it does not attempt to re-connect and thus also stops updating the underlying orderbook).
I have "solved" this issue in my app by re-connecting the orderbook whenever the last update is older than few minutes. However, it would be better if this was fixed in the master branch.

  constructor(this._client) {
    this._client.on("close", data => this.onStateChange());
  }

  private onStateChange = () => {
    if (!this._client.socket) setTimeout(() => this._client.connect(), 5000);
  };

@ctubio Mind opening a PR?

why this issue([bug]) is open ?

Closing this due to inactivity. Please open a new Issue and/or PR (possibly referencing this issue) if further action is deemed necessary.

repo: coinbase/gdax-node


text:
[storage] Protect against nil Helper and/or Handler in BalanceStorage

Someone was able to force a panic in rosetta-cli because of an unpopulated BalanceStorage helper and handler. This PR causes BalanceStorage to raise an error in these cases instead of panicking.
Changes

[x] Add new error
[x] Check for nil helper or handler

Pull Request Test Coverage Report for Build 13577

7 of 21 (33.33%)  changed or added relevant lines in 1 file are covered.
402 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.1%) to 77.981%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

storage/modules/balance_storage.go
7
21
33.33%

Files with Coverage Reduction
New Missed Lines
%

storage/modules/coin_storage.go
15
80.0%

asserter/network.go
21
86.22%

storage/modules/counter_storage.go
21
46.32%

storage/modules/broadcast_storage.go
63
70.72%

storage/modules/balance_storage.go
141
77.79%

storage/modules/block_storage.go
141
76.37%

Totals

Change from base Build 13567:
-0.1%

Covered Lines:
8078

Relevant Lines:
10359

💛  - Coveralls

authors: coveralls patrick-ogrady

repo: coinbase/rosetta-sdk-go


text:
Enable automatic connect/reconnect...

I'm not sure what the limitations are preventing the strategy @colinbdclark had in mind for managing (re)connection as devices are connected and disconnected.
My thought is to use the same functionality that powers the --list-ports option, and to have an outer "connection manager" component whose model has the available inputs and outputs.  Whenever the available inputs and outputs change, it would recreate the router instance, which would be bound to an appropriate createOnEvent option.  The "connection manager" would poll the ports intermittently (say every 15 seconds) and update its model with the latest available inputs and outputs.
The "connection manager" would stay alive indefinitely, so we'd need to display instructions about how to abort it ("hit ctrl-c to exit...").  If no matching inputs and outputs are found, the router throws an error and aborts as before.  If new devices are connected, a new router will be given the chance to wire things up.

The issues are a bit subtle, but I'll try to summarize here:
At the moment, Flocking's MIDI implementation on Node.js instantiates a new set of Port objects for all connected input and output ports each time flock.midi.nodejs.getAllPorts() is invoked. This is due to limitations in node-midi and its underlying RTMidi library, which provides no stable way for uniquely identifying ports across all platforms.
However, node-midi and RTMidi have a bug, which the maintainers have not responded to, in which native RTMIDI Port objects are not correctly released when they are no longer being used. This means that, using its current strategy and libraries, Flocking leaks memory whenever Ports are accessed but not used. Combined with polling, this causes ALSA to eventually crash once too many ports have been allocated. This is documented in Flocking #197.
The new, unfinished flocking-midi library addresses this by only working with modelized representations of ports until the user actually connects to a port. This, in combination with a hand-assembled fork of node-midi that I created, ensures that polling the MIDI system won't cause crashes.
However, flocking-midi isn't yet ready for prime time, in that it doesn't yet implement the ability to connect to ports via model-bound dynamic components. This is coming, and crucially needed, but I haven't yet had an opportunity to finish the implementation.
One solution might be to look into mixing the polling functionality of flocking-midi with the current implementation built into Flocking? It could be complex, and may require changes, but it might work.
Note, also, that none of these issues are present in the browser. The Web MIDI API provides a much better interface for accessing and creating ports, including events that fire whenever a port is added or removed. Much of the problem on Node.js stem from the fact that RTMidi has a very poor interface that makes it hard to enumerate ports and does not provide stable port identification across platforms; issues have been filed and pull requests submitted, but it seems not much has been done about it.
Does this help?

authors: colinbdclark the-t-in-rtf

repo: colinbdclark/flocking-midi-router


text:
TeX Extension Options

Implement TeX extension options.

Hi @colinc86!
Great work in the library. Could you guide me on how I can help with this? I would like to add support for some AMSmath labels like \binom

Hey thanks, @juancazalla, and sorry for not getting around to this sooner!
You can load different extensions through the TexInputProcessorOptions' loadPackages property. Only the base extension is loaded by default, so this will error:
let output = try await mathjax.tex2svg("\\binom{n}{k}")

But this will render:
let texOptions = TeXInputProcessorOptions(loadPackages: [TeXInputProcessorOptions.Packages.ams])
let output = try await mathjax.tex2svg("\\binom{n}{k}", inputOptions: texOptions)

If you want all of the extensions available to MathJax loaded you can use TeXInputProcessorOptions.Packages.all as the input array.


text:
.toLowerCase/UpperCase() should infer to Lowercase/Uppercase<string>

The Lowercase and Uppercase types are relatively new to TS and already have analogs in Zod.
When I do this:
const MySchema = z.string().toLowerCase();
type MyType = z.infer<typeof MySchema>;

I should get this:
type MyType = Lowercase<string>

In zod v3.21.4, I get this:
type MyType = string

Hey 👋
Despite z.string() always infers to string (and not template literal types), this actually makes sense.

Lowercase<string> & Uppercase<string> extend string, so it shouldn't add any complexity to downstream inference.
You can actually force inputs of a function to be lowercase / uppercase:

function fn<T extends Lowercase<any>>(str: T) {}

fn('moshe') // OK!
fn('Moshe') // type-error!

So having this in zod, removes the need for type-guards before passing str to the function.

TypeScript collapses Lowercase<string> to just string. The Lowercase built-in is intended for converting string literal types.
type MyType = Lowercase<string>
// => string

authors: brianblakely colinhacks igalklebanov

repo: colinhacks/zod


text:
Patient dashboard patial: Make Last menstrual period human readable and editable

We currently have a top partial that looks like this:

I think it's an unresolved question of how we want to display last menstrual period so that it's human readable, and I'm not sure I have a great understanding of how it goes in the crypto spreadsheet. Let's hash it out here. For context, here's the invision prototype: https://projects.invisionapp.com/share/6757W6WFJ#/screens/115983266
Here's two options:

Figure out some way to accept either a date or a week for LMP in a single form field, and convert it on the backend (I think this might match the current workflow more closely?)
Change last menstrual period date to date of first contact, keep last menstrual period weeks as it is

(Other options are also good!)
attn @NerdyGirl537  @lwaldsc -- seeking your thoughts as our client experts.

Duplicate issue; closing. (Also resolved in #211 )


text:
you dont need to call commit after remove_all!

Hi, why do you need to invoke Sunspot.commit after Sunspot.remove_all!?
remove_all! will invoke commit https://github.com/sunspot/sunspot/blob/ec64df6a526d738f9f77c039679b344f908d3298/sunspot/lib/sunspot/session.rb#L188

Hi there! We’re doing some open source maintenance work and closing old issues. If you would still like this issue to be addressed, please re-create it by May 17, 2017. Thanks!

authors: asheren sevaorlov


text:
Notify Internal Slack when a thread gets 100+ replies 😅

Issue Reserved for Bloomberg Mentorship Program
Description
Sometimes things get a little hectic in the family slack..😅 In the case that a single Slack thread gets 100+ replies, we want the admins to get a notification so that they can monitor the thread in case it is getting out of hand.
Acceptance Criteria

Implement a function within ‘packages/core/src/modules/slack/queries' that counts the number of replies a thread has (this will require counting how many times that particular threadId appears in the database)
This function should also check if the number of messages in the thread is exactly 100, in which case the function should send a notification to the internal slack with a message implying that a particular thread is getting long
This function should be called within the addSlackMessage function after adding a new message to the database

Additional Context
We have a function addSlackMessage  that retrieves any new Slack message and adds it to the database (see here). Within this function, after adding the message to the database, we can trigger a new function that will count the number of replies in the given thread and notify the admins if there are exactly 100 messages in the thread.
To notify the admins, we can utilize the job notification.slack.send similar to as used here. Note that all we have to do is put the workspace as ‘internal’ and add a message. Make sure to specify which thread is getting long.

I’ll take it his

authors: ciaracade tomas-salgado

repo: colorstackorg/oyster


text:
Fixes: #3550 ; Add hello-world Android Java Example using Mill

Description
This PR fixes #3550 by adding a "hello-world" Android Java example using Mill.
Changes:

Created 1-hello-world folder inside example/javalib/android

This folder contains a src folder for sources and a build.mill file.

Created two modules:

AndroidSdkModule.scala: This module handles the SDK setup.
AndroidAppModule.scala: This module extends both AndroidSdkModule and JavaModule to ensure proper functionality.

In build.mill, extended the App module for building the Android application.
android_examples.adoc created and referred in navbar.adoc, finally tested.
Documentation done for android modules and build.mill and finally tested
Code is fully documented with references taken from example/{javalib,kotlinlib,scalalib} and scalalib/{Modules...}.
All the Documentation is checked locally (worlking as expected for scaladoc{AndroidAppModule, AndroidScalaModule})
Documentation and explanation of the code are embedded as comments within the code.
Included a screenshot of the application to demonstrate its functionality.

Type of Change

[x] New feature (non-breaking change which adds functionality)

Related Issues
issue #3550
Testing
Tested by building the Android app using Mill, ensuring that both the AndroidSdkModule and AndroidAppModule work together as expected. Successfully generated the APK and verified the "Hello World" functionality.
Checklist

[x] Created 1-hello-world folder with build.mill and sources
[x] Created two modules AndroidAppModule.scala and AndroidSdkModule.scala
[x] Created documentation file android_examples.adoc, above two modules also done.
[x] Code builds and passes all tests.
[x] Documentation Tested with local doc's creation
[x] I have added comments in the code where necessary, updated the documentation accordingly.
[x] Any dependent changes have been merged and published.

Additional Notes
Feel free to review the code and let me know if any further improvements are needed.
Screenshots

@lihaoyi Sir please don't get irritated by my doubts(i am newbie) please clear my last doubt...
basically when i reviewed Json files(allSources.json, allSourceFiles.json .....so on) in the out folder:
allSources.json

{
    "value": [
        "ref:v0:c984eca8:/home/himanshu138/opensource/himanshu/mill/out/example/javalib/android/1-hello-world/local/testCached.dest/sandbox/run-1/app/src"
    ],
    "valueHash": -2002708993,
    "inputsHash": 1715811750
}

allSourceFiles.json

{
    "value": [],
    "valueHash": 1467096776,
    "inputsHash": 435515105
}

so after this i came to know that even JavaModule is unable to pick up my src files because it is also get path added with app becoz of which i am getting error of no source files and then i checked all previous examples/javalib/ and found this :
object `package` extends RootModule with JavaModule {...}

and when i tried this same thing then i got shocked this time it got the files and sources, see:
what i tired
object `package` extends RootModule with AndroidAppModule {...}

allSources.json
{
    "value": [
        "ref:v0:49c3a842:/home/himanshu138/opensource/himanshu/mill/out/example/javalib/android/1-hello-world/local/testCached.dest/sandbox/run-1/src"
    ],
    "valueHash": -1645131398,
    "inputsHash": 1044320490
}

allSourceFiles.json
{
    "value": [
        "ref:v0:60186141:/home/himanshu138/opensource/himanshu/mill/out/example/javalib/android/1-hello-world/local/testCached.dest/sandbox/run-1/src/main/java/com/helloworld/app/MainActivity.java"
    ],
    "valueHash": -249522556,
    "inputsHash": -470604943
}

Final Verdict: we also need to use package and make androidApp as a RootModule object
please share your thoughts and suggestions...

Yes the filesystem hierarchy has to match the module hierarchy

Yes the filesystem hierarchy has to match the module hierarchy

Thanks Alot Sir finally now i got answers to all of my doubts...

@lihaoyi Thankyou Sir for the detailed feedback!
I’ve addressed all the requested changes and resolved the comments. Please let me know if anything else needs attention.
Please pardon me if I missed anything or made a mistake. Looking forward to your final review and any further suggestions!

Thanks @himanshumahajan138 , I think it looks pretty good overall, let me try running some end-to-end tests on an android emulate on my laptop and then we can close this out

Thanks @lihaoyi Sir It's You who gave me another chance to show up my skills and contribute to Mill.
I hope all the tests pass successfully🤞

Tested locally on my android emulator, works great on my macbook. A few final comments and we can merge it.

That's great Sir I will do the required changes and push the final code asap
Thanks Alot @lihaoyi  Sir

@lihaoyi Sir Finally It's done, Thank you so much sir for your help and support

Looks good, merging it and we can tweak it more in main. @himanshumahajan138 can email me your international bank transfer details and I can send you the bounty

@lihaoyi Sir I have sent the details to your email mentioned in your GitHub account.
Please Confirm the details...

repo: com-lihaoyi/mill


text:
tests: add tests to RoleFilter component

Closes: issue-52

References: /
What?

Add tests for RoleFilter component

Why?

To ensure the component is returning null or an given element depending if user has or not proper roles

How to test it?
# to run all tests suits
yarn test

# press 'a' to run all tests and see tests for RoleComponents passing

Not quite sure if what I've done on useRoleFilter tests is the more appropriated approach. Following the documentation of @testing-library/react-hooks in this section, to render hooks that needs information from some provider(e.g redux), we need to use a wrapper, and this wrapper needs to be simple as:
const wrapper = ({ children }) => <CounterStepProvider step={2}>{children}</CounterStepProvider>

But our wrapper returns a rendered element and didn't work as expected. If you guys have a tip to improve my tests I will be thankful

Not quite sure if what I've done on useRoleFilter tests is the more appropriated approach. Following the documentation of @testing-library/react-hooks in this section, to render hooks that needs information from some provider(e.g redux), we need to use a wrapper, and this wrapper needs to be simple as:
const wrapper = ({ children }) => <CounterStepProvider step={2}>{children}</CounterStepProvider>

But our wrapper returns a rendered element and didn't work as expected. If you guys have a tip to improve my tests I will be thankful

I've pushed a commit refactoring the wrapper component to support hooks.

authors: Isaac-alencar belgamo

repo: comarev/comarev-dashboard


text:
about example data

I would like to ask if I don't seem to find the subset tag in the sample data, only 'celltype_major' and 'celltype_minor'.
In addition, results from other methods are provided in the data, and I found that the different methods predicted a large disparity in the number of classes. However, these methods are based on marker data for classification, and removing the effect of unknown types, they should predict the same number of classes, right? I hope to get your help.

Hi
'celltype_major' and 'celltype_minor' are the manual annotation. They were created from the original manual annotation in 'celltype' column in obs field. I just renamed them to compare with HiCAT results.
HiCAT results are stored in 'HiCAT_major', 'HiCAT_minor' and 'HiCAT_subset', where the first two can be compared with the manual annotations to evaluate accuracy.
Regarding the question "However, these methods ...., right?", Yes, exactly, because I used the same markers (at different level) for all the tools I ran.
If you want to run other annotation tools I used, you can use the R wrapper package RCTI_collection at https://github.com/combio-dku/RCTIcollection

authors: combio-dku nbnbhwyy

repo: combio-dku/HiCAT