mail gem breaks attachment file with encoding Quoted-Printable.

I found this issue when I encode mail with binary file attachment in Quoted-Printable.
This is sample case.
require 'mail'

# 1. Create Mail with Quoted-Printable
mail = Mail.new
part = Mail::Part.new
bin = File.binread("./some_binary.jpg")
part.body = [bin].pack("M")
part.content_transfer_encoding = "quoted-printable"
part.transport_encoding = Mail::Encodings::QuotedPrintable
mail.parts << part
File.binwrite("./tmp/original.eml", mail.to_s)

# 2. Read mail and write

mail = Mail.read("./tmp/original.eml")
File.binwrite("./tmp/after.eml", mail.to_s)

This is difference between original.eml and after.eml.
diff --git 1/tmp/original.eml 2/tmp/after.eml
index 7d2c038..506176e 100644
--- 1/tmp/original.eml
+++ 2/tmp/after.eml
@@ -27,15 +27,16 @@ Content-Transfer-Encoding: quoted-printable
 =01=01=00=02=03=00=00=00=00=00=01=02=11=03=12!1=04A=05=13Q"a2q=15=81=B1=FF=
 =DA=00=0C=03=01=00=02=11=03=11=00?=00=1C|f=98=08=A2=C6i@=E2=BFW?9=07=B2A=14=
 =80=F7=A2=04=E4=FB=D2  =13=C5=02=07=B7=A5"=93=9A =1C=D2=8E=B4=0D=02=03=9E=94=
-=FBD=CCQ=12=93=CD8=14=08=08H=02=9Bl=8C=1A6=DC=F1L=13=8A=00=1E=DEim=10{
+=FBD=CCQ=12=93=CD8=14=08=08H=02=9Bl=8C=1A6=DC=F1L=13=8A=00=1E=DEim=10{=0D=
+
 ,sM=1C=98=A4=00=80=A5=B6=8B=B6=96=DA=07=E8=06=DAr=98=E9E=00=FBR=DB=88=A0@=
 =B6=E3=8C=D3G?=D6=8B=B78=A4=A0&=80=05=1C=D3m=A2m=A5=B6=80=07=B4SE=16&i=14=
 =FD(=001=DA=91O=D6=8AS=8E=0D4`=E6=90=D2=03=B6:R=83=EDE=8C=D3m=89=14=C0=19=
-Gz=8CGJ)=039=A6=89=C4=D0=82=81=ED=00qM=00=0E1E =C6M4b=90P";
+Gz=8CGJ)=039=A6=89=C4=D0=82=81=ED=00qM=00=0E1E =C6M4b=90P";=0D
 b=9Fl=D1H=3D)=B6=CF\=F5=A0(=14w=A6=DBE"=04=E6=A2S@=03=81=C7=EBM=B4DQ=00=C4=
 R=8A=07@=8AzTv=E4=D1J}=A9=A0=CD=00=08=A4v8=A8=C7<=D1=A2=A2S#=8A=01=A0[y=A8=
 =C1=E2=8D=B7=EBLA=C9=14=82=80=ED D=7F=D8=D4v=FE=94h"=98=8Fj`=90=18=A6#"=8D=
-=1E=D5=10=07JV:=04=13Q=8C=9A)=07=E1M=04=0CM =02=A4=8C=FB=D4
+=1E=D5=10=07JV:=04=13Q=8C=9A)=07=E1M=04=0CM =02=A4=8C=FB=D4=0D
 z=8A6=DE=D4=D1=88=E9@#=AA      =81=EFNR>=B4M=A7=A5 =9CI=AB%=03=DA =E2=96=DFj$=01=
 Kn S=00a<=C6iG1=D6=88=13=8CR=03=BD =06=13=CC=D2=DB=EDD#=9A@c=A5=00=0C=8AQ=

This problem cause from ruby Array#pack("M") behavior.
Ruby's Array#pack("M") doesn't support Quoted-Printable with binary, so ruby does not encode "\n".
["\n"].pack("M") # => "\n"

and Mail library converts "\n" to "\r\n" in to_crlf. so break binary file.
I found similar issue in #1010
I asked to ruby community. (https://bugs.ruby-lang.org/issues/14352 sorry this discussion in Japanese.) and I got a answer which this behavior should not be change.
so I propose to change implementation in Mail gem's Quoted-Printable encoder as follow.
diff --git i/lib/mail/encodings/quoted_printable.rb w/lib/mail/encodings/quoted_printable.rb
index 193778c..6a04833 100644
--- i/lib/mail/encodings/quoted_printable.rb
+++ w/lib/mail/encodings/quoted_printable.rb
@@ -16,11 +16,18 @@ module Mail
       # Decode the string from Quoted-Printable. Cope with hard line breaks
       # that were incorrectly encoded as hex instead of literal CRLF.
       def self.decode(str)
-        str.gsub(/(?:=0D=0A|=0D|=0A)\r\n/, "\r\n").unpack("M*").first
+        str.unpack("M").first
       end

       def self.encode(str)
-        [str].pack("M")
+        [str].pack("M").gsub(/(=0D\n|(?<!=)\n)/) do |match|
+          case match
+          when "=0D\n"
+            "=0D=0A=\r\n"
+          when "\n"
+            "=0A=\r\n"
+          end
+        end
       end

       def self.cost(str)

This change has breaking change, breaks test case. For example.
Failures:

  1) quoted-printable should encode quoted printable from text
     Failure/Error: expect(Mail::Encodings::QuotedPrintable.encode("This is\na test")).to eq result

       expected: "This is\na test=\n"
            got: "This is=0A=\r\na test=\n"

       (compared using ==)

       Diff:
       @@ -1,3 +1,3 @@
       -This is
       +This is=0A=
        a test=

This change is compatibility will be lost, but more safety than before.
any idea?
Thanks.

Hi,
I can confirm the bug and the proposed solution works for me.
https://www.ietf.org/rfc/rfc2821.txt
SMTP commands and, unless altered by a service extension, message
data, are transmitted in "lines".  Lines consist of zero or more data
characters terminated by the sequence ASCII character "CR" (hex value
0D) followed immediately by ASCII character "LF" (hex value 0A).
This termination sequence is denoted as <CRLF> in this document.
Conforming implementations MUST NOT recognize or generate any other
character or character sequence as a line terminator.  Limits MAY be
imposed on line lengths by servers (see section 4.5.3).

In addition, the appearance of "bare" "CR" or "LF" characters in text
(i.e., either without the other) has a long history of causing
problems in mail implementations and applications that use the mail
system as a tool.  SMTP client implementations MUST NOT transmit
these characters except when they are intended as line terminators
and then MUST, as indicated above, transmit them only as a <CRLF>
sequence.

@mikel do you have time to review?

I prepared a pull request and fixed some issues https://github.com/mikel/mail/pull/1246
but 2 issues still remain:
1/ all lines should always contain only  separators
2/ max line length should be always <= 76 or separated by a soft line break "=" (I don't have a real problem reading it, but it violates RFC now)
I tried to fix them, but it breaks binary attachments again. Any idea?

I'm also having this issue @mikel


Feature: Support Catalog User Interface

Description
Support the VulcanSQL Catalog user interface for retrieving data in a more efficient and user-friendly manner.
Sample vulcan.yaml
Please notice auth options should be either basic or password-file is setting up or disabled the auth
name: playground1
description: A starter Vulcan project
version: 0.1.0-alpha.1
template:
  provider: LocalFile
  # Path to .sql files
  folderPath: sqls
  codeLoader: InMemory
artifact:
  provider: LocalFile
  serializer: JSON
  # Path to build result
  filePath: result.json
schema-parser:
  reader: LocalFile
  # Path to .yaml files
  folderPath: sqls
document-generator:
  specs:
    - oas3
types:
  - RESTFUL
extensions:
  duckdb: '@vulcan-sql/extension-driver-duckdb'
profiles:
  - profile.yaml
rate-limit:
  options:
    interval:
      min: 1
    max: 10000
enforce-https:
  enabled: false
auth-source:
  options:
    key: token
auth:
  enabled: true
  options:
    basic:
      htpasswd-file:
        path: ./passwd.txt
        users:
          - name: jax
            attr:
              department: engineer
          - name: shimin
            attr:
              department: engineer
    password-file:
      path: ./bcrypt-passwd.txt
      users:
        - name: jax
          attr:
            department: engineer
        - name: shimin
          attr:
            department: engineer
response-format:
  enabled: true
  options:
    default: json
    formats:
      - json
      - csv

Catalog Server
Run Develop Command
yarn nx serve catalog-server

open http://localhost:4200
UI

Catalog list

Catalog detail & preview dataset result

Connect with third-party services tutorial

VulcanSQL Serve
Run Develop Command
In labs/playground1, run
make start
// or
vulcan start // if already build

open http://localhost:3000

Add /auth/available-types for retrieving auth type by API
Add catalog-routers for providing APIs to Catalog Server.

/catalog/schemas
/catalog/schemas/:sourceName
The schema APIs above will additionally provide url: string, apiDocUrl: string, shareKey: string

url: host + urlPath in VulcanSQL
apiDocUrl: the API document url in VulcanSQL
shareKey: the query string to pass the auth middleware

exclude API docs path in auth middleware

VulcanSQL CLI
After discussion, the package.json will  need to be updated after catalog publishing
You will need to do the actions below if you want to test in local
In labs/playground1

Please add the line in package.json

{
   "dependencies": {
       ...
       "@vulcan-sql/catalog-server": "../../dist/packages/catalog-server"
   }
}

Do make to install all packages

You can use vulcan catalog command line in another terminal after running the VulcanSQL server with vulcan start.

VulcanSQL Package

add -t --target for specific which target need to build and the default is vulcan-server

For Catalog Server, you can add the command below
vulcan package --output node --target catalog-server

if user want to adjust the package folder path, can setting it like below

node-packager:
   vulcan-server:
       folderPath: dist
   catalog-server:
       folderPath: dist
docker-packager:
   vulcan-server:
       folderPath: dist
   catalog-server:
       folderPath: dist

Additional Context

Codecov Report
Patch coverage: 56.55% and project coverage change: -1.01 :warning:

Comparison is base (d80a255) 92.97% compared to head (eaed70f) 91.96%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #160      +/-   ##
===========================================
- Coverage    92.97%   91.96%   -1.01%
===========================================
  Files          289      295       +6
  Lines         4398     4518     +120
  Branches       582      592      +10
===========================================
+ Hits          4089     4155      +66
- Misses         200      252      +52
- Partials       109      111       +2

Flag
Coverage Δ

build
95.01% <ø> (ø)

catalog-server
100.00% <100.00%> (?)

cli
85.79% <48.00%> (-3.60%)
:arrow_down:

core
94.00% <ø> (ø)

extension-dbt
97.43% <ø> (ø)

extension-debug-tools
98.11% <ø> (ø)

extension-driver-bq
85.21% <ø> (ø)

extension-driver-duckdb
100.00% <ø> (ø)

extension-driver-pg
96.11% <ø> (ø)

extension-driver-snowflake
94.59% <ø> (ø)

integration-testing
96.15% <ø> (ø)

serve
87.17% <56.98%> (-2.87%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/serve/src/containers/types.ts
100.00% <ø> (ø)

...kages/serve/src/lib/auth/httpBasicAuthenticator.ts
87.23% <ø> (ø)

...es/serve/src/lib/auth/passwordFileAuthenticator.ts
81.81% <ø> (ø)

...ges/serve/src/lib/auth/simpleTokenAuthenticator.ts
100.00% <ø> (ø)

...ges/serve/src/lib/catalog-router/catalogRouters.ts
24.39% <24.39%> (ø)

...ve/src/lib/middleware/auth/authRouterMiddleware.ts
91.48% <33.33%> (-8.52%)
:arrow_down:

packages/cli/src/commands/catalog.ts
43.47% <43.47%> (ø)

...c/lib/middleware/auth/authCredentialsMiddleware.ts
84.61% <77.77%> (-0.39%)
:arrow_down:

...kages/serve/src/models/extensions/catalogRouter.ts
80.00% <80.00%> (ø)

.../middleware/auth/authSourceNormalizerMiddleware.ts
90.90% <83.33%> (-1.95%)
:arrow_down:

... and 9 more

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Hi @fredalai
I have fixed all the suggestions above :D
Please review again when you have free time, thanks~


🐛 Bug Report: RequirePermission causes "Mounted routable extension must have a path"

📜 Description
When attempting to wrap routes in RequirePermission as follows:
      <Route path="/route-here" element={
        <RequirePermission permission={myPermissionHere}>
          <MyComponentHere />
        </RequirePermission>
      } />

I am encountering the following error:
Error
Mounted routable extension must have a path
Call Stack
 Object.eval [as visit]
  node_modules/@backstage/core-app-api/dist/index.esm.js:1981:15
 eval
  node_modules/@backstage/core-app-api/dist/index.esm.js:1752:42
 eval
  node_modules/react/cjs/react.development.js:1104:17
 eval
  node_modules/react/cjs/react.development.js:1067:17
 mapIntoArray
  node_modules/react/cjs/react.development.js:964:23
 mapChildren
  node_modules/react/cjs/react.development.js:1066:3
 Object.forEachChildren [as forEach]
  node_modules/react/cjs/react.development.js:1103:3
 traverseElementTree
  node_modules/@backstage/core-app-api/dist/index.esm.js:1744:49
 eval
  node_modules/@backstage/core-app-api/dist/index.esm.js:2648:24
 mountMemo
  node_modules/react-dom/cjs/react-dom.development.js:15846:19

This error does not occur when removing the RequirePermission parent tag.
I would be happy to submit a PR for this, but I'm not sure where the problem is.
👍 Expected behavior
Successfully render the application, displaying the component when the permission check passes and displaying an error page when it doesn't.
👎 Actual Behavior with Screenshots

👟 Reproduction steps
Introducing a RequirePermission tag into any Route tag element within packages/app/src/App.tsx causes the above error to occur.
📃 Provide the context for the Bug.
No response
🖥 Your Environment
Browser: Chrome Version 107.0.5304.121 (Official Build) (x86_64)
yarn backstage-cli info:
yarn run v1.22.19
$ /Users/dan.hoizner/git/collibra/portal/node_modules/.bin/backstage-cli info
OS:   Darwin 22.1.0 - darwin/x64
node: v16.15.1
yarn: 1.22.19
cli:  0.20.0 (installed)
backstage:  1.7.0

Dependencies:
  @backstage/app-defaults                          1.0.7
  @backstage/backend-common                        0.15.2, 0.16.0, 0.17.0
  @backstage/backend-plugin-api                    0.1.4
  @backstage/backend-tasks                         0.3.7
  @backstage/catalog-client                        1.1.2
  @backstage/catalog-model                         1.1.3
  @backstage/cli-common                            0.1.11
  @backstage/cli                                   0.20.0
  @backstage/config-loader                         1.1.7
  @backstage/config                                1.0.5
  @backstage/core-app-api                          1.1.1
  @backstage/core-components                       0.11.2
  @backstage/core-plugin-api                       1.2.0
  @backstage/dev-utils                             1.0.7
  @backstage/errors                                1.1.4
  @backstage/integration-react                     1.1.5
  @backstage/integration                           1.4.1
  @backstage/plugin-api-docs                       0.8.10
  @backstage/plugin-app-backend                    0.3.37
  @backstage/plugin-auth-backend                   0.17.0
  @backstage/plugin-auth-node                      0.2.8
  @backstage/plugin-catalog-backend                1.5.1
  @backstage/plugin-catalog-common                 1.0.8
  @backstage/plugin-catalog-graph                  0.2.22
  @backstage/plugin-catalog-import                 0.9.0
  @backstage/plugin-catalog-node                   1.2.1
  @backstage/plugin-catalog-react                  1.2.0
  @backstage/plugin-catalog                        1.6.0
  @backstage/plugin-github-actions                 0.5.10
  @backstage/plugin-org                            0.5.10
  @backstage/plugin-permission-backend             0.5.14
  @backstage/plugin-permission-common              0.7.2
  @backstage/plugin-permission-node                0.7.2
  @backstage/plugin-permission-react               0.4.8
  @backstage/plugin-proxy-backend                  0.2.31
  @backstage/plugin-scaffolder-backend             1.7.0
  @backstage/plugin-scaffolder-common              1.2.2
  @backstage/plugin-scaffolder                     1.7.0
  @backstage/plugin-search-backend-module-pg       0.4.1
  @backstage/plugin-search-backend-node            1.0.3
  @backstage/plugin-search-backend                 1.1.0
  @backstage/plugin-search-common                  1.1.1
  @backstage/plugin-search-react                   1.2.0
  @backstage/plugin-search                         1.0.3
  @backstage/plugin-tech-radar                     0.5.17
  @backstage/plugin-techdocs-backend               1.4.0
  @backstage/plugin-techdocs-module-addons-contrib 1.0.5
  @backstage/plugin-techdocs-node                  1.4.1
  @backstage/plugin-techdocs-react                 1.0.5
  @backstage/plugin-techdocs                       1.3.3
  @backstage/plugin-user-settings                  0.5.0
  @backstage/release-manifests                     0.0.6
  @backstage/test-utils                            1.2.1
  @backstage/theme                                 0.2.16
  @backstage/types                                 1.0.2
  @backstage/version-bridge                        1.0.3
Done in 0.87s.

👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

I just went through the steps in the react-router stable migration again and the issue is resolved, sorry!
Hopefully this helps anyone else that runs into this in the future.
Closing!

I just went through the steps in the react-router stable migration again and the issue is resolved, sorry! I must have missed a step the first time around.
Hopefully this helps anyone else that runs into this in the future.
Closing!

In the following section of the doc,

Step 1 - Upgrade to Backstage 1.6
The first Backstage release to support react-router v6 is 1.6. You should upgrade to this version first before you start migrating. If you are an early bird and want to try out migration before that release, it is also shipped in 1.6.0-next.1.

I assume this means Backstage v1.16 and v1.16.0-next.1?


Error "SyncMirrors [...] unable to write commit-graph [...]" on mirroring

Description
Just noticed  https://github.com/nextcloud/contacts mirror sync error:
2023/03/29 08:08:31 ...irror/mirror_pull.go:279:runSync() [E] [6423d5db-14] SyncMirrors [repo: 38:nc/nextcloud-contacts]: unable to write commit-graph for '/myreporoot/nc/nextcloud-contacts.git' : exit status 128 - error: Could not read 544c4227b061eb5331617430a3896872fd3915ab
2023-03-29    08:08:31    gitea[123]: error: Could not read a3a65b5a36972ed4c50d461e9fe6127a66c2ea05
2023-03-29    08:08:31    gitea[123]: error: Could not read 681e22178c7cefeb3033740f2cbb93312a615fc9
2023-03-29    08:08:31    gitea[123]: error: Could not read 24df40d4cf028ba93ff68f7230e059dce1374c2f
2023-03-29    08:08:31    gitea[123]: error: Could not read 82fb27f04da07758971fff6f9d99d30e21f61295
2023-03-29    08:08:31    gitea[123]: error: Could not read 46ae973a9d8c6abf9b07da4ce0a27014c275f459
[...]
2023-03-29    08:08:31    gitea[123]: error: Could not read e4b86229e61575ab45542e6d337ddf3892a78f4f
2023-03-29    08:08:31    gitea[123]: error: Could not read 03a21dd37f6f01da2d637f789baa4066271d1080
2023-03-29    08:08:31    gitea[123]: fatal: unable to parse commit 03a21dd37f6f01da2d637f789baa4066271d1080
2023-03-29    08:08:31    gitea[123]: - error: Could not read 544c4227b061eb5331617430a3896872fd3915ab
2023-03-29    08:08:31    gitea[123]: error: Could not read a3a65b5a36972ed4c50d461e9fe6127a66c2ea05
2023-03-29    08:08:31    gitea[123]: error: Could not read 681e22178c7cefeb3033740f2cbb93312a615fc9
[...]
2023-03-29    08:08:31    gitea[123]: fatal: unable to parse commit 03a21dd37f6f01da2d637f789baa4066271d1080
2023-03-29    08:08:31    gitea[123]:

Mirror was created many months ago and worked fine (everyday sync) till today. Manual resync throws same error in this mirror.
Commit 544c4227b061eb5331617430a3896872fd3915ab mentioned in errors is present in github but not in mirror (looks like orphaned on github).
Fresh https://github.com/nextcloud/contacts mirror setup on other system with same gitea+git versions does not throw such error on sync.
Gitea Version
v1.18.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.30.2
Operating System
No response
How are you running Gitea?
Compiled from sources.
Database
MySQL

A quick google search says that git fsck or git gc might help. It looks like some git internal data corrupted (or out of sync)

Running Garbage collect all repositories manually from Monitoring resolved the issue, thank you.
git gc is fired automatically only during some operations, so probably is not fired on mirror update and that caused the problem. Please consider enabling cron.git_gc_repos by default to avoid such surprises. gc and fsck jobs should probably have longer timeout by default (1800s maybe?) to handle bigger repos on slower hardware by default.

Or maybe we can change the issue's title to proposal a default value change?

Done.

Concurrent use of git gc and some git commands may lead to data corruption. I don't know if it is related to what is happening here<EMAIL_ADDRESS>

Concurrent use of git gc and some git commands may lead to data corruption.

To avoid problems consider:

adding warning to cron.git_gc_repos docs about corruption possibility with link to mentioned page,
introducing checkbox in every repo config to explicitly allow cron.git_gc_repos to process it (similar to existing Enable Repository Health Checks (git fsck) checkbox).

With such checkboxes admin may enable cron.git_gc_repos only for repos they treat as non-critical and safe (i.e. read only mirrors).


fix UI feedback

Closes https://github.com/stencila/sibyl/issues/10 except for https://github.com/stencila/sibyl/issues/44. Thanks!
Screenshot

Codecov Report

Merging #45 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   79.54%   79.54%
=======================================
  Files           1        1
  Lines         176      176
=======================================
  Hits          140      140
  Misses         36       36

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46cc49c...c093681. Read the comment docs.

@yoshuawuyts : LGTM. There's a conflict in the bundle.js. Could you resolve and merge.


[feature]gMSA v2 support on Windows AKS-HCI

Current version of gMSA will require the nodes to be in a domain. gMSAv2 is not required domain joined.

This feature is completed. Public documentation is available at https://docs.microsoft.com/en-us/azure-stack/aks-hci/prepare-windows-nodes-gmsa.


Update trace4cats-fs2, trace4cats-testkit to 0.12.0-RC1+188-05746b3c

Updates

io.janstenpickle:trace4cats-fs2
io.janstenpickle:trace4cats-testkit

from 0.12.0-RC1+156-7ea07b63 to 0.12.0-RC1+188-05746b3c.
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.janstenpickle" } ]

labels: library-update, semver-build-metadata

Superseded by #5.


Create /team page and show all the contributers there

Description
Create /team page and show all the contributers there
Screenshots
No response
Additional Information
No response

Yes we are working on design of team page, do you like to contribute in design (figma)?

I would like to develop the page I am not designer

We'll let you know about it :)
Happy contributing

Hey as a contributor of GSSOC'23 ,I want to work with the project .Assign me the issue

I want to work on this issue
Pleasure Assign so that I can show my skills under GSSoC'23.

@AbhiPatel10 @himanshu1221 is the design ready ?

@AbhiPatel10 @himanshu1221 is the design ready ?

Not yet, Work in progress

Hi, I'd like to help. Could I get assigned to this issue ? Thank you !

Can you please assign me this issue under GSSOC'23?

Guys basic layout of teampage is ready once it live i will ask you to make changes on that page

closing this issue as this issue has been resolved


Quick access to theme change slider

The Issue
Many users are big fans of Dark mode, but some users may not even know it exists. We can try exposing this setting on the bottom left of the sidebar, and alert users to it as a first run hint. My suggestion would be to make dark mode default for the next named release (D, for Dark as a double meaning on the author name), along with the prompt so users can turn back to light mode easily.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Will be in the settings bar on next release.


reading parquet file fails

reading a parquet file gives error:
Failed to get a valid column type for integer type 17

Not sure, but 17 seems to be:
https://github.com/duckdb/duckdb/blob/16830ded9b4882b90f35c2d7e2d740547ae7d3fc/src/include/duckdb/common/types.hpp#L197
TIMESTAMP_SEC

happens as well for a csv
file, find attached.
(def ds  (duckdb/sql->dataset conn "SELECT * FROM read_csv_auto('flights.csv')"))

flights.csv.gz

is fifex in latest commit


Fix Turing MH sampler

I noticed that I had used the wrong syntax for the MH sampler in Turing in #323. I wanted to use the same random walk proposal as for the mini turing example but currently it uses static Normal(0, 1) proposals. This explains also the low ESS.
It seemed easiest (and probably it's one of the most efficient approaches) to just specify the covariance matrix of the random walk proposals, and it should be more efficient to specify explicitly that it is an identity matrix. Alternatively, but IMO less concise and/or efficient, one could use MH(:a => x -> Normal(x, 1), :b => x -> Normal(x, 1)) or MH(:a => AdvancedMH.RandomWalkProposal(Normal()), :b => AdvancedMH.RandomWalkProposal(Normal())).

Now the ESS estimates are similar: https://julialang-buildkite-artifacts.s3.amazonaws.com/turingtutorials/6f982592a7399e505fe00b53509b4a75a0d920f13033c375dbda8539d62d2299/html/14-minituring/14_minituring.html

It seemed easiest (and probably it's one of the most efficient approaches) to just specify the covariance matrix of the random walk proposals

That is true for this simple example, which is indeed a two-dimensional Gaussian. However, I don't think we need to be optimal with the sampler here -- the main purpose is to illustrate how the DSL works.
Many thanks for the quick fixes and sorry that I didn't catch it in #323.


Cross-platform online, can pc and android ios play games together?

Please explain the suggested feature in detail.
Can pc android ios connect to the same server at the same time?
i want use mirror to make a Multi-platform VR Game
Android IOS PC play games together

Probably the wrong place to ask this question as it's not an issue. Please use the Discord in the future. 😄
But yes, you can use mobile device clients with desktops and consoles. Mirror can handle that. How you implement that the cross-platform functionality is up to you as the programmer. Mirror just gives you the fundamental for the networking, you build upon it.

Probably the wrong place to ask this question as it's not an issue. Please use the Discord in the future. 😄
But yes, you can use mobile device clients with desktops and consoles. Mirror can handle that. How you implement that the cross-platform functionality is up to you as the programmer. Mirror just gives you the fundamental for the networking, you build upon it.

Thank you, I just want to know that mirror's api can handle multi-platform interoperability, and I will do the upper-layer platform adaptation.
For other questions I go to Discord


Removed sample links from fundamentals

Addresses #1102

Thank you for this contribution. We have left #1102 on the backlog because we'd like to hold off on removing the sample links until we can get more customer feedback and data on whether readers find them useful or not. Closing this PR for now.


Purify get functions

Most of the getter functions in the library are not pure. They depend on which user is loaded in the state and restrict the functionality of the relay REST API.
Refactor getter functions to be pure, i.e.
// not pure
tlNetwork.trustline.get(network, counterpartyAddress)

// pure
tlNetwork.trustline.get(network, user, counterpartyAddress)

But the clientlib is not just a lib for easy querying the relay server. We should only make that change if it is not harder for the normal user to use the lib.

The user of the clientlib would have to additionally pass in one more parameter in the getter functions. Would you consider this as harder to use?
@jensneuber what is your opinion on this?

Needs to be handled within a bigger issue that we need to create to handle refactoring of the client lib. If that issue is created this issue can be closed.


utils/transcripts.py for chunking preserving previous behaviour as default for #8087

fixes #8087
added chunking mechanism for for transcript loaders srt.py and youtubeloader.py so user could optionally load transcript for a certain duration

Hey @baskaryan  @eyurtsev and @rlancemartin why were some checks not sucessfull? any idea of what can I do?
Thank you!

Hey @baskaryan @eyurtsev and @rlancemartin why were some checks not sucessfull? any idea of what can I do?
Thank you!

looks like some linting errors. you can run make lint from the libs/langchain directory to see linting issues locally! and make format to automatically fix any formatting issues

@beaterblank Hi , could you, please, resolve the merging issues and address the last comments (if needed)? After that, ping me and I push this PR for the review. Thanks!

Hey sure yeah, thank you!

@leo-gan Hi, the merge issue was regarding updating the branch, and I have updated my branch,
regarding previous comments I was told that

all configuration information is passed via the initializer by convention

so I addressed that in my previous commit removing duration from load function and I've put it in the initializer as per the convention

@eyurtsev Please, review it. Thanks

@eyurtsev please review, Thanks.

any update on this please? @beaterblank

any update on this please? @beaterblank this is so cool if successfully merged!

@w4rlock999, I'm waiting for it to be reviewed  and merged, I made the requested changes still waiting to be merged.


Update DOCUMENTATION.md

Updated example Spotify song to something decent.

LGTM 🚀

WITH PREJUDICE.

Is PREJUDICED


Permissions Issue

Love the image! Small issue when trying to use it - seems there's a permissions issue in the latest one?
postgres_1  | FATAL:  data directory "/var/lib/postgresql/data" has group or world access
postgres_1  | DETAIL:  Permissions should be u=rwx (0700).

I believe you're missing this in your entrypoint somewhere. I don't have the time to debug it myself but figured I'd leave a note - would be cool to see it patched up!
chmod 700 "$PGDATA"

Apologies, seems GitHub had a hiccup and this posted twice!


Invalid parameter when trying to add Applet

I tried to upload log.wasm file from https://github.com/machinefi/w3bstream/blob/main/_examples/log/log.wasm
I am unable to add it and below is response from server.
{
    "key": "badRequest",
    "code": 400000000,
    "msg": "invalid parameters",
    "desc": "",
    "canBeTalk": false,
    "id": "",
    "sources": [
<EMAIL_ADDRESS>    ],
    "fields": [
        {
            "field": "info.wasmName",
            "msg": "missing required field",
            "in": "body"
        }
    ]
}

I managed to reproduce the issue. When trying to upload a "hello world" example from the documentation with the code
// log.go
package main

import (
	"unsafe"
)

// main is required for TinyGo to compile to Wasm.
func main() {}

func _log(logLevel uint32, strPtr uint32, strSize uint32)

func _start(rid uint32) int32 {
  	ptr, size := stringToPtr("Hello World!")
  	const LOG_LEVEL_INFO = 3
  	_log(LOG_LEVEL_INFO, ptr, size)
  	return 0
}

func stringToPtr(s string) (uint32, uint32) {
	buf := []byte(s)
	ptr := &buf[0]
	unsafePtr := uintptr(unsafe.Pointer(ptr))
	return uint32(unsafePtr), uint32(len(buf))
}

and compiling it with tinygo build -o log.wasm -scheduler=none --no-debug -target=wasi log.go
When I modify the docker-compose to get around the CORS issue (to be able to log in - without this modification, the w3bstream studio will fail to log in due to CORS issue) by adding
w3bstream-studio:
    image: ${WS_STUDIO_IMAGE:-ghcr.io/machinefi/w3bstream-studio:stable}
    links:
      - "w3bapp:localhost"
    .....

to the w3bstream-studio definition, I can log in, but when attempting to upload the file I get this error:

from the w3bstream docker logs:
   "@agent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:107.0) Gecko/20100101 Firefox/107.0",
   "@cst":5,
   "@lv":"warning",
   "@mtd":"POS",
   "@prj":"srv-applet-mgr",
   "@rmt":"<IP_ADDRESS>",
   "@status":400,
   "@tag":"access",
   "@ts":"20221205-170124.627Z",
   "@url":"/srv-applet-mgr/v0/applet/9011896374453256",
   "msg":"{\"key\":\"badRequest\",\"code\":400000000,\"msg\":\"invalid parameters\",\"desc\":\"\",\"canBeTalk\":false,\"id\":\"\",\"sources\":[\"srv-applet-mgr@0.0.1\"],\"fields\":[{\"field\":\"info.wasmName\",\"msg\":\"missing required field\",\"in\":\"body\"}]}\n"
}```

What version of W3bstream are you using? i.e., did you run the official docker image with
curl https://w3bstream.com/dc  > docker-compose.yaml && docker-compose up -d

or are you building from the latest commit?

@simonerom The w3bstream docker-compose doesn't work properly in regards to networking. Without modifying the source of the docker-compose.yaml to edit the following line:
NEXT_PUBLIC_API_URL: ${NEXT_PUBLIC_API_URL:-http://w3bapp:8888}
to be
NEXT_PUBLIC_API_URL: ${NEXT_PUBLIC_API_URL:-http://localhost:8888}
You cannot even log into the portal. It's due to CORS issues

I tried the link you gave, plus the instructions on the github repo, as well as the instructions on the official documentation. None of them work. They all have the same issue.
If you make the changes to localhost you can log in, but importing the wasm file fails also, might be due to CORS issue as well, I don't know

Hello @simonerom @jrynkiew,
It's official docker image, What I did was just add below env variable in .env file and ran docker-compose up -d
NEXT_PUBLIC_API_URL=http://server_ip_address:8888
it was not running properly without this env variable as it was sending request form w3bstream studio to http://w3bapp:8888 by default

"w3bapp" should be resolved to the container's IP address using docker-compose default networking, but it's most likely failing due to CORS issues - the w3bstream-studio container is trying to make a call to an "external" URL, and default internet security settings by default block "cross-origin" calls. The individual containers, even though they are on one host machine, refer to each other by their domain names, which is interpreted by the browser as being an external source of data, and is blocked. The w3bstream studio must refer to other container endpoints using the host mapped ports, or it will fail. Or the networking cors strict policy rules must be modified

@yalamber @simonerom There is a workaround this issue. Please don't use the main master branch when building the w3bstream modules.
do this instead:
git clone https://github.com/machinefi/w3bstream.git
cd w3bstream
git checkout tags/v0.9.4
make init_submodules
make build_docker_images
make run_docker

Now when you go to http://localhost:3000/ you should be able to log in and upload your wasm file
Best Regards,
IoTeX team


Replace halo2 pinned commit w/ released version

When published to crates.io.

Duplicate of #2982.


FoundationPress doesn't seem to be working with ajaxify (AWS plugin)

Hello,
I am trying for the second time to use ajaxify wordpress site on a FoundationPress theme.
Problem is, if there is anything using javascript (for example my menu is using the sticky plugin from Foundation), it's not working after an ajax reload. Which is weird, because the foundation.js file is being reloaded...
Has anyone made this theme work with ajax or is it just me ?
Here is the example of the not working website if it can help anybody : https://lesdeuxvagues.com/demo/
EDIT :
I have found the root of my issue.
Most of the javascript in foundation.js is executed on the document.ready event.
The problem is that with an ajax load, there is no document.ready event.
I have found that there is no way to simulate it.
Would it possible to create a new function, for example onPageLoad(), where all of this code goes, and this one function would be called on document.ready. And us people working with ajax could call this function again on ajaxComplete ?
Would be nice !

Someone ? :)

I don't see this as a bug in FoundationPress, but rather with the integration of a third party plugin. I will therefore close the issue.


React Native 0.40 Compatibility

Is this repo supporting version 0.40?

Not yet. But I send a PR that fix this. Wait for merge #78.

What else is needed to support version 0.40?

Have you solved this problem? not supprot RN 0.4>= Version?

It looks like @lucianomlima solved the iOS part, so we're left with Android. Anyone can make this happen?

I installed manually this on RN 0.40 and it worked with Android, I just changed the location of the interterstial id.

@fcpauldiaz Location of interstitial on iOS or Android too?

@fcpauldiaz @lucianomlima since sbugert seems inactive, can one of you test a working version of 0.40 and fork it? Others and future users can benefit from your work, since there seems to be no other admob repo.

@lucianomlima only on Android, I haven't tested it on iOS. @edencakir I can do an example over the weekend. If someone needs an example right now, you can check my app repo just be careful with the production ads.

@fcpauldiaz you can send me the code you change?
Maybe I try to update my pull request to fix compatibility for Android too.

I based all my code from the Example from this repo. The lines that I actually changed are this, this,  this, this and this.
And of course the versions of RN.

@fcpauldiaz thanks, I've seen your code and apparently the interstitial issue is not directly related to the RN 0.40+ version.
So, maybe Android platform don't need adjusts for RN 0.40+.

That's great! So you can make a release now

Here in this repo I can't, cause I'm not collaborator.

i just typed commands.
version
"dependencies": {
"react": "15.4.2",
"react-native": "0.41.2",
"react-native-admob": "^1.1.5"
},
npm i react-native-admob -S
react-native link
pod install
target 'admob' do
Uncomment the next line if you're using Swift or would like to use dynamic frameworks
use_frameworks!
pod 'Firebase/AdMob'
Pods for admob
target 'admob-tvOSTests' do
inherit! :search_paths
# Pods for testing
end
target 'admobTests' do
inherit! :search_paths
# Pods for testing
end
end
Build fali since then. How do you solution with it?

Do you have a working fork? @lucianomlima

@YoonJaePark see #78 or use RN 0.39

@edencakir my fork only have a branch with the changes I do in PR #78
Here I'm still using RN 0.39

Closed by #95

Published https://github.com/sbugert/react-native-admob/releases/tag/v1.2.0
Please feel free to ping me if anything is wrong.


[Q] It is possible to override json file driver globalAdditionalCapabilities from code ?

When using selenoid is possible to set TestName as global capability Selenoid Custom Test Name. Is it possible to override this key from code to set different test name for each test case ?
Thanks,

Basically, you need to invoke AddAdditionalCapability("name", AtataContext.Current.TestName) for WebDriver options. You can try something like this:
AtataContext.Configure().
    UseRemoteDriver().
        //...
        WithOptions<ChromeOptions>().
        WithOptions(x => x.AddAdditionalCapability("name", AtataContext.Current.TestName)).
        //...

Please share your AtataContext initialization code, if it doesn't work.

I'm use json config to initialize and want to set current test name as AdditionalCapability.

Thanks,

OK. Please try this way:
AtataContext.GlobalConfiguration.
    ApplyJsonConfig(@"...json").
    UseDriver(DefaultBrowserAlias);

var remoteDriverBuilder = AtataContext.GlobalConfiguration.BuildingContext.DriverFactoryToUse as RemoteDriverAtataContextBuilder;

if (remoteDriverBuilder != null)
    remoteDriverBuilder.WithOptions(x => x.AddAdditionalCapability("name", AtataContext.Current.TestName));

Looks good, but do not work =)
Because it is GlobalAdditionalCapability and should pass as
x.AddAdditionalCapability("name", AtataContext.Current.TestName, global : true)
But overloaded method with isGlobal parameter is missed in abstract DriverOptions class.

In this library code you used fabric to define concrete DriverType  that provide possibility to set AdditionalCapability as global.
Thanks,

OK, got it. Let's try to do it this way:
AtataContext.GlobalConfiguration.
    ApplyJsonConfig(@"...json").
    UseDriver(DefaultBrowserAlias);

var remoteDriverBuilder = AtataContext.GlobalConfiguration.BuildingContext.DriverFactoryToUse as RemoteDriverAtataContextBuilder;

if (remoteDriverBuilder != null)
{
    remoteDriverBuilder.WithOptions(options =>
    {
        string capabilityName = "name";
        string capabilityValue = AtataContext.Current.TestName;

        (options as ChromeOptions)?.AddAdditionalCapability(capabilityName, capabilityValue, true);
        (options as FirefoxOptions)?.AddAdditionalCapability(capabilityName, capabilityValue, true);
        (options as InternetExplorerOptions)?.AddAdditionalCapability(capabilityName, capabilityValue, true);
        (options as OperaOptions)?.AddAdditionalCapability(capabilityName, capabilityValue, true);
    });
}

If it also doesn't work for you, please share your driver JSON configuration to see the whole picture.

Currently working with
AtataContext.GlobalConfiguration.
    ApplyJsonConfig(@"...json").
    UseDriver(DefaultBrowserAlias);

var remoteDriverBuilder = AtataContext.GlobalConfiguration.BuildingContext.DriverFactoryToUse as RemoteDriverAtataContextBuilder;

if (remoteDriverBuilder != null)
{
   remoteDriverBuilder.WithGlobalCapability("name", "TestName");
}

Glad that you found that new WithGlobalCapability method.
You know, I was also thinking to provide the functionality to use some templated variables in JSON for TestName and probably some others:
"globalAdditionalCapabilities": {
  "name": "{test-name}"
}

I guess it will be useful, keeping all configuration in JSON.


is it working properly?

is it working properly?
it only says Server is listening ...
and believe nothing more happen

s


Missing tags from VT comments

munin only collects the tags of VT itself, not from the comments, but they're displayed in the comments line:
HASH: 4512e183370cd0fa8175b03692682d408811bf8afa70f11d3517de823e2adb9b RULE: -
VIRUS: ...
TYPE: HTML SIZE: 189.75 KB FILENAMES:
FIRST: 2023-01-12 02:26:55 LAST: 2023-01-12 06:09:22 SUBMISSIONS: 2 REPUTATION: 0
COMMENTS: 1 USERS: thor TAGS: HTML CONTAINS-EMBEDDED-JS

question is: what's the best way to fix it? just put VT and comment tags together in vt-hash-db.json or create a new field e.g. comment_tags?
      "tags": [
            "html",
            "contains-embedded-js"
        ],

        "comments": 1,
        "commenter": [
            "thor"
        ],

API doc:
https://developers.virustotal.com/reference/comments

fixed in https://github.com/Neo23x0/munin/pull/61


Readme and Ruby require

I just noticed a small issue in the Readme.md and documentation.
When trying to use the gem as specified in the example:
# Ruby
require 'gherkin3'

I get an error "`require': cannot load such file -- gherkin3 (LoadError)". Replacing it by:
require 'gherkin3/parser'

works like a charm. I prefer not to make a pull request, I ain't sure this comes from our project configuration or a real issue in the documentation.
Cheers.

Thanks for the heads up!


Update ngrok to latest

This URL actually points to the latest version of ngrok and has for
every release after 2.0.19 from around March 2016.

I've verified this information using the Internet Archive (https://web.archive.org/web/20160304144547/https://ngrok.com/download) and by reviewing commits over the past year for this formula.


Google service - "This browser or app may not be secure"

Describe the bug
When logging in to a Google service it doesn't allow on the basis of being a non-secure app.
To Reproduce
Steps to reproduce the behavior:

Add Gmail as Service
Insert username and password
See error

Desktop (please complete the following information):

OS:  Windows 10 Home - 10.0.18363 Build 18363
Version 5.3.3

I have the same issue!

Same issue: MacOS Mojave (version 10.14.6)

Same issue on Linux Mint 19.2 (Franz ver. 5.4.0)

Same issue here for GMail and Google Calendar.
MacOS Catalina 10.15.1
I tried to re-install Franz from scratch but this didn't help.

Same issue here, too, but only for one of the Google accounts I'm trying to use in Franz. Another account has no such issues.
Windows 10
Version: 5.3.3
Release: 10.0.18362

Same issue. Fedora 31, Franz 5.3.3.

same here with Franz 5.4.0-beta on macOS Catalina

Google recently banned few linux browsers, marking them as not secure. Most likely the underlying browser that Franz is using behind the scenes is the one of them.

Hangouts don't work  -> "This browser or app may not be secure"
MacOS Catalina 10.15.1 & Franz <IP_ADDRESS>2

Changing the user agent works for me.

Close Franz
open /home/jann/.config/Franz/recipes/hangouts/index.js in a text editor
as the last line add:

module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

Yeah @q-g-j it's works! But.. I've other path:
~/Library/ApplicationSupport/Franz/recipes/hangouts/index.js

Same issue, Franz  5.4.1  and MacOS Catalina 10.15.2.

For Windows, the location of the file is: C:\Users[username]\AppData\Roaming\Franz\recipes[google_service]\index.js

Same issue for my gmail and google calendar, I have spent over an hour with google support, we have tried everything possible and came to the conclusion that its a problem with the application

I needed a different url
~/Library/ApplicationSupport/Franz/recipes/gmail/index.js
But potentially gmail needs to replaced by the google service you need.
My file now looks like this:
"use strict";

//module.exports = Franz => Franz;
module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

We are seeing this in Ferdi as well and reckon it would be easier for each user to be able to override the user agent in the service settings as a more sustainable solution:

As seen in #263, #229 and #140, Google (and maybe other services) are regularly making changes to prevent usage of their services within some browsers/apps. It seems this detection is based on the user agent. If we had remote update of services (still pending via getferdi/server#14), we could update the user agent used in the recipe to periodically overcome this, but ultimately the updated user agent will be somewhat blocked. This is why it would be useful for each user to be able to override the user agent used by a specific service in the service settings, so each user can use whatever user agent works for them (and avoid using the same user agent for all Ferdi user via a recipe update).

Via https://github.com/getferdi/ferdi/issues/264

Same issue here! Please fix.

Same issue here with Franz 5.4.1 and Ubuntu 19.10.

Same issue with Franz 5.4.0 and Ubuntu 18.04.3 LTS

Changing the user agent works for me.

Close Franz
open ~/.config/Franz/recipes/hangouts/index.js in a text editor
as the last line add:

module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

That's an Edge-Browser for which Google always shows warnings to support Google Chrome.
I'd use your usual browsers user agent.
On Windows the file path is %AppData%\Franz\recipes
someone should update the chrome engine in Franz to close this issue.

Changing the user agent works for me.

Close Franz
open ~/.config/Franz/recipes/hangouts/index.js in a text editor
as the last line add:

module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

That's an Edge-Browser for which Google always shows warnings to support Google Chrome.
I'd use your usual browsers user agent.
On Windows the file path is %AppData%\Franz\recipes
someone should update the chrome engine in Franz to close this issue.

This solved to me!

As of today, this issue seems to have been fixed? Perhaps Google whitelisted the Franz user agent?
I'm running 5.3.3 on Windows 10 Pro.

Still problematic on mine. 5.4.1 on Mac Catalina 10.15.1

Weirdly enough it only seems to be on login. I have the same issue with GMail and Google Calendar, I made this change to the gmail index.js and it let me log in. But then if I clicked any links in emails it said it had been blocked by a popup blocker. I closed franz and removed it, when I reopened I can use gmail as usual. So I guess google is only checking the user agent at log in 🤦♂

Yes, that check is only at login. However, if you quit Franz and didn’t have to login, it’s probably because you didn’t fully quit Franz. If you’d fully quit, you'd almost certainly have to login again.

Huh, what? I fully quit Franz all the time without having to log back into my services

However, if you quit Franz and didn’t have to login, it’s probably because you didn’t fully quit Franz. If you’d fully quit, you'd almost certainly have to login again.

Not really, you are also not required to login to gmail every single time you quit your browser.

Changing the user agent works for me.

Close Franz
open ~/.config/Franz/recipes/hangouts/index.js in a text editor
as the last line add:

module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

In my case the problem was with the Todoist app, so if you guys are with the same problems with other apps just find the app folder under ~/.config/Franz/recipes and then paste the code above changing the class (normally is the app name in CamelCase).
So i did:

Close Franz
open ~/.config/Franz/recipes/todoist/index.js in a text editor
as the last line add:

module.exports = Franz => class Todoist extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

see this reddit thread on how to fix the Electron app
https://www.reddit.com/r/electronjs/comments/eiy2sf/google_blocking_log_in_from_electron_apps/
app.userAgentFallback = app.userAgentFallback.replace('Electron/' + process.versions.electron, '');

Same issue here, but the module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };  fix did not work for me

Same issue here, but the module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } }; fix did not work for me

It only worked for me after I quit franz and restarted it.  Simply disabling the service and enabling it again after making the change did not seem to work but the module.exports line does seem to be a fix at the moment!

Appending Edge/12.10136  seems indeed to fix the issue, but most of the URLs in mails now open a popup instead of a new tab

Provided workaround not working for me on Windows =(
After applying different user agent it loads mobile site version and when I click Sign in, redirects to external browser.

This is the same issue as #1573. Closing one or the other would be helpful.

I've noticed it only happens in some accounts but not all. I have another Google account and didn't this error message while my private one did.

Same problem on Ubuntu 18.04.3 LTS and Franz 5.4.0. The solution above worked for me in the services where I had problems (Google Calendar and Trello). Hope this is fixed soon

Me too - different application (IM+).
Not secure... or maybe blocks ad serving? You decide.

After several unsuccessful attempts, I managed to get my Gmail service back working again. I imagine this is the case with any Google service. This is what I did to get it working on MacOS 10.14.6.

Open ~/Library/Application\ Support/Franz/recipes/gmail/index.js
Change the line: module.exports = Franz => Franz; to be the following:

module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

Quit the entire Franz app
Reopen Franz

And then it just started working for me. Hopefully that might help someone else who is trying to get it to work.

same here. Manjaro KDE

@zboman solution works for me on Linux Mint. Thanks man!

@zboman
Thank you for you solution, but it doesn't work for me.
As others have mentioned, using my Chrome agent causes my browser to open the login page instead of Franz.
Using MacOS 10.15.3 and Franz 5.4.1

having same issue on macOS Catalina

Changing the user-agent works for me. I'm using Manjaro Linux and Franz 5.4.1.
What I did is edit the file ~/.config/Franz/recipes/ and change the file index.js as follows:
`
// Comment out the original setting
//module.exports = Franz => Franz;
// Add this setting. It seems only Windows user agent can work
module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 6.1; WOW64; rv:54.0) Gecko/20100101 Firefox/73.0"; } };
`
For some reason **only when the OS is Windows ** will this work for me.

Same problem here: Windows 10 Education, freshly downloaded and installed franz. Please fix :/

I've noticed it only happens in some accounts but not all. I have another Google account and didn't get this error message while my private one did.

I am having this exact same error. Two of my Google accounts work fine, and the third does not.

Happens with Google Voice as well

Same issue here.
OS: MacOS 10.15.3 (19D76)
Franz: 5.3.3 (<IP_ADDRESS>2)
Changing the useragent did not solve the issue for me.

Same issue here.
OS: MacOS 10.15.3 (19D76)
Franz: 5.3.3 (<IP_ADDRESS>2)
Changing the useragent did not solve the issue for me.

For me neither.

Describe the bug
When logging in to a Google service it doesn't allow on the basis of being a non-secure app.
To Reproduce
Steps to reproduce the behavior:

Add Gmail as Service
Insert username and password
See error

Desktop (please complete the following information):

OS:  Windows 10 Home - 10.0.18363 Build 18363
Franz: Version 5.4.0

It worked for me
just copy the code pasted by kayintveen and Press control+R to open Run and type in Run %Appdata% and enter and then go from there to franz and recipes and in my case i can't login into gmail and get in gmail folder and open index with notepad ++ or just right click and select edit option in menu and it opened with windows own notepad and delete all of them and paste this in there
"use strict";
//module.exports = Franz => Franz;
module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };
in my case it was gmail in your case it might be different if it is then change the class gmail name and put your app which is not working and press control+S to save close all and open Franz and see if its working

Someone may be assigned to this issue, to update chrome engine in Franz

https://github.com/meetfranz/franz/issues/1771#issuecomment-606446211
#1771

This on worked for me on MacOS Catalina :
Edited ~/Library/Application Support/Franz/recipes/gmail :
"use strict";
module.exports = Franz => class useragent extends Franz
{
    overrideUserAgent ()
    {
        return "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36";
    }
};

Links opens correctly, no side effects 👍
Important : I needed to remove my Gmail tab and recreate a new because the old tab was unable to connect.

Nothing of those worked for me, any other solutions? win 10
module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.130 Safari/537.36"; } };

module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

Got correct values from http://www.browser-info.net updated in place of module.exports = Franz => Franz; and still same error.

what should i use for the class name for google voice?

Changing user agent fixed my multiple Gmail account issues, but I am unable to log into Google voice with any user agent I try. There is something different for Google voice.
Same Google mail account works with modified user agent and does NOT work with the same user agent for Google voice.

Solved for with me with Franz update 5.5.0-beta.2.1747

Fixed for me as well with Franz update Version 5.5.0-beta.2 (5.5.0-beta.2.1747), on OSX Catalina 10.15.4.   Thank you! 🙏(The previous mentioned solutions did not work for me.)

But how do I use the beta version, there is no exe for that!

This on worked for me on MacOS Catalina :
Edited ~/Library/Application Support/Franz/recipes/gmail :
"use strict";
module.exports = Franz => class useragent extends Franz
{
    overrideUserAgent ()
    {
        return "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36";
    }
};

Links opens correctly, no side effects 👍
Important : I needed to remove my Gmail tab and recreate a new because the old tab was unable to connect.

This worked for me on Windows 10, Franz version 5.4.0. I changed the class to useragent.

Changing the user agent works for me.

Close Franz
open ~/.config/Franz/recipes/hangouts/index.js in a text editor
as the last line add:

module.exports = Franz => class Hangouts extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };

Hello,
Thank you for sharing. I do not understand how to apply for this IT language. I tried to open Terminal in my Macbook pro and copy + paste which doesn't work. I have the issue with the Gmail and G calendar. Would you be able to help me, please?
Thanks very much in advance!

Describe the bug
When logging in to a Google service it doesn't allow on the basis of being a non-secure app.
To Reproduce
Steps to reproduce the behavior:

Add Gmail as Service
Insert username and password
See error

Desktop (please complete the following information):

OS:  Windows 10 Home - 10.0.18363 Build 18363
Franz: Version 5.4.0

It worked for me
Press control+R to open Run and type in Run %Appdata% and enter and then go from there to franz and recipes and in my case i can't login into gmail and get in gmail folder and open index with notepad ++ or just right click and select edit option in menu and it opened with windows own notepad and delete all of them and paste this in there
"use strict";
//module.exports = Franz => Franz;
module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };
in my case it was gmail in your case it might be different if it is then change the class gmail name and put your app which is not working and press control+S to save close all and open Franz and see if its working

Dosen't work for me!
Im using Win10, 64bits, just this account I cant loggin my Gmail on Franz, anothers accounts dosnt appears this issue.
I dont know what to do!

Does anyone know what to change the string to for Google Voice? Because it just deletes module.exports = Franz => class Google Voice extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36"; } }; when I launch the service.

same issue with win10 x64

6 months since there is this issue with Franz and nobody fixes it. Disappointing to see they add paid features instead.

Same pb after password change, recipe fix does not work either

For me the update process didn't work. So I deinstalled the software and
reinstalled the new one which worked fine.
Yann Guégan<EMAIL_ADDRESS>schrieb am Sa., 30. Mai 2020, 16:26:

Same pb after password change, recipe fix does not work either
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/meetfranz/franz/issues/1720#issuecomment-636338183,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH3HJ5HLDQYNSU6Y4SVMRCTRUEJQBANCNFSM4J2DOVDA
.

same for me. i added login using google. and it works fine except that i see this error message when start a debugger in vs code. .... can not continue a development

just did a clean installation using the version https://chocolatey.org/packages/franz delivers (should be 5.5.0) and got the problem.
Interesting enough - on the same machine on a Windows 10 on a different SSD the problem doesn't appears.
Used Googlebot User Agents to prevent it...

I used Franz 5.4.0 and updating UserAgent to Chrome/42.. fixed this problem for me in "hangoutschat" recipe.
Here is content:
module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };
Just upgraded to 5.5.0 and it stopped to work with message "This browser does not support security keys."
Recipe source code is the same. Looks like something was changed in Franz itself. I see some changes related to UserAgent in this commit  (https://github.com/meetfranz/franz/commit/d7840ba) and I suspect that workaround that changes UserAgent doesn't work anymore.
Can someone help with question - how can I check what UserAgent is used when try to login to Google in "hangoutschat" recipe?  Doe it use overridden value "Chrome/42.0.2...." or default one?
I have Win 7, Franz 5.5.0

I updated to the latest beta version (5.6.0-beta 1) and I was able to log into Hangouts!

I just tried to upgrade to (5.6.0-beta 1) and it still doesnt work. It doesnt ask me to insert security key and new error message is
Your sign-in settings don't meet your organization's 2-Step Verification policy.
Contact your admin for more info

Does anyone know what to change the string to for Google Voice? Because it just deletes module.exports = Franz => class Google Voice extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36"; } }; when I launch the service.

This worked for me on OSX

Describe the bug
When logging in to a Google service it doesn't allow on the basis of being a non-secure app.
To Reproduce
Steps to reproduce the behavior:

Add Gmail as Service
Insert username and password
See error

Desktop (please complete the following information):

OS:  Windows 10 Home - 10.0.18363 Build 18363
Franz: Version 5.4.0

It worked for me
Press control+R to open Run and type in Run %Appdata% and enter and then go from there to franz and recipes and in my case i can't login into gmail and get in gmail folder and open index with notepad ++ or just right click and select edit option in menu and it opened with windows own notepad and delete all of them and paste this in there
"use strict";
//module.exports = Franz => Franz;
module.exports = Franz => class Gmail extends Franz { overrideUserAgent() { return "Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10136"; } };
in my case it was gmail in your case it might be different if it is then change the class gmail name and put your app which is not working and press control+S to save close all and open Franz and see if its working

Works for me!


Sirap prerelease/v0.14.0 sirap prerelease.0

I closed this PR because the sirap-prerelease branches will never be merged in master.
They lives while the upstream/master release is tagged.


Pure-pwsh

A new implementation using only PowerShell (no C#)

hmmm I have to think of the best way to tackle these merge conflicts in a non-painful way 😄


EMQX

Hello fellas, this plugin works to connect to EMQX Broker for IoT data streaming?

@synapsedata I know, late reply, but since I was working on something over the Holidays with this datasource, I can confirm this is working very well with EMQX. My only problem right now is support for dynamic topic as requested here:

https://github.com/grafana/mqtt-datasource/issues/63

Without dynamic topic, you're dashboard wil be statically bound to certain topic only.


 Now it doesn't fetch all the contracts for the "default user" unless you actually don't have enough for yourself.

FYI @jahooma.


How to dismiss a promise toast from its own button

Hello,
I have a toast promise that I'd like to dismiss from the button it shows upon completion. How could I acomplish that?
Here is a minimal example:
import toast, { Toaster } from "react-hot-toast";

const muhpromise = () => {
  return new Promise(async (resolve, reject) => {
    setTimeout(() => {
      resolve();
    }, 500);
  });
};

export default function App() {
  return (

 {
          const myPromise = muhpromise();
          toast.promise(myPromise, {
            loading: "Loading",
            success: (

                I'd like to dismiss here
                <button onClick={() => alert('how do I do that?')}>Dismiss</button>

            ),
            error: "Error when fetching"
          });
        }}
      >
        Hello CodeSandbox

Start editing to see some magic happen!

      <Toaster />

  );
}

You can also find the codepen here.
Thank you for your time!

Thank you very much. You have yourself a wonderful day.


Add a RELEASING document

This document details the steps to complete to release a version of pyIATI.

This adds many of the steps to undertake before starting to release a version on PyPi.
This section needs filling in since I'm not entirely sure on the steps.
(the rest of it also needs proofing)

Something else:

Update pyIATI to include the latest relevant version of SSOT content.

@dalepotter @allthatilk Thoughts on where Update pyIATI to include the latest relevant version of SSOT content. should go?

Marking as incomplete until I've run through the "Release to PyPi" steps.

I've now run through the "Release to PyPi" steps. Several tweaks have been made in the process.


Prevent sites being indexed by search engines

Is your feature request related to a problem? Please describe.
I want to prevent some sites to be indexed by search engines.
Describe the solution you'd like
A checkbox to prevent a site being indexed or the option to add custom headers like: add_header  X-Robots-Tag "noindex, nofollow, nosnippet, noarchive";
Describe alternatives you've considered
Adding the header by adding a location block in the custom configuration but it requires me to write my own location block configuration. That I don't understand completely I think because I can't load some sites afterwards.
location / {
    proxy_pass http://<IP_ADDRESS>:8123;
    add_header  X-Robots-Tag "noindex, nofollow, nosnippet, noarchive";
}

Additional context
I hope is is possible to add this feature and I think a lot of people are interested in it.

Definitely need this. Any workarounds until someone gives an update?
Do we have to write every single proxy_pass for every subdomain?

Looking for this option too 🙂

Would also love to have this

would agree it would be great to have this

any updates on this

Start a bounty. I'll pitch in for this feature.

This would be very useful


Clarify hostNetwork and address other outstanding feedback

Address outstanding feedback and clarifications from https://github.com/kubernetes/website/pull/36197 after it was merged.

@ddebroy, great blog. You have aptly described a new status as part of the pod lifecycle. A casual and/or new blog reader might find the text-only presentation challenging. A simple diagram would improve reader accessibility.
You can use Mermaid to create diagrams as described in the Diagram Guide. Check out K8s system flow diagrams as examples, Mermaid sequence diagram doc or use labeled boxes with arrows. Up to you. Keep it simple.
You can address this in another PR. Tx for researching and authoring this helpful blog. Looks fine to publish.

/lgtm
/approve


Add VSCode

Reasons for making this change:
VSCode support in Unity 5.5.0
Links to documentation supporting these rule changes:
https://unity3d.com/unity/whats-new/unity-5.5.0

Thanks, but see Global\VisualStudioCode.gitignore


Uninstall the setuptools that is present on Semaphore nodes

There is loads of stuff pre-installed on a Semaphore ubuntu2004 node, including Python stuff that is:

distribution-installed (apt-get), in /usr/...

sudo pip installed, in /usr/local/...

pip installed in various specific Python virtual environments.

Yesterday there was an update to that - https://docs.semaphoreci.com/reference/semaphore-changelog/ - and that has caused our OpenStack Yoga testing to fail.  Removing the pre-installed /usr/local/ setuptools brings it back to green again.

I'm sorry but I failed delete the branch after merging the pull request.


Uncaught TypeError: $timeout is not a function

Uncaught TypeError: $timeout is not a function
at Object.onFinish (angular.ion.rangeSlider.js:105)
at IonRangeSlider.callOnFinish (ion.rangeSlider.js:1158)
at IonRangeSlider.pointerUp (ion.rangeSlider.js:626)
at dispatch (jquery-2.2.2.min.js:3)
at r.handle (jquery-2.2.2.min.js:3)

Hey @mikila85, I pushed on my repository this bug fixed. I was not allowed create pull request in this repository.

I just merged your branch into mine @WesleyMarques


[Backend] adding a field of name for brand and  name , mobile no store  and category for influencer in their respective db models without disturbing other functionalities

make changes in db models both brand and influencers and add a field of brand name and for influencers add name , mobile no  and category.

@RohitGupta1237  can you please assign this to me


Closes #5232

Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[ ] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

Put "Closes #5232" in the PR description so it actually links to it, and write a short title about this PR request instead (ex. "CosplayTele: Update popularMangaRequest" or whatever)


SyntaxError on Generic

Attempting to utilize the Generic module since I need to try this with Telegram. I am getting a Syntax Error coming from the library itself (using Python 3.9):
Traceback (most recent call last):
  File "app.py", line 4, in <module>
    from botanalytics.generic import Generic
  File "/env/lib/python3.9/site-packages/botanalytics/generic.py", line 9
    callback=None, async = False):

Here is my usage:
botlogger = Generic(debug=True, token=API_KEY)

...
 botlogger.log({
        "user": user.id,
        "is_sender_bot": False,
        "message": message,
    })

Hey, ran into this mysef.
It seems the problem is the  callback=None, async = False):
async is an forbidden variable name in Python. So if you change it from async to async_ it should work.
change to:
callback=None, async_ = False):

Hey, ran into this mysef.
It seems the problem is the  callback=None, async = False):
async is an forbidden variable name in Python. So if you change it from async to async_ it should work.
change to:
callback=None, async_ = False):

Hi, just ran into the same problem. This should be fixed. It's a naming issue in the package itself. I can't make my code automatically deploy with this bug

Thank you for reporting this issue.
I just committed a fix and will release the new version on PYPI.


Sample codes to create a mosaic does not work (TypeScript, JavaScript)

Describe the error
Hi, I followed the following tutorial to create a mosaic and executed the JS and TS samples, but neither worked for me.
https://nemtech.github.io/guides/mosaic/creating-a-mosaic.html
network type is MIJIN_TEST.
I thought there was something wrong with my catapult server at first, but it wasn't.
I could create a mosaic with nem2-desktop-wallet, connecting to the local node.
I compared the source code I used and one of nem2-desktop-wallet.
https://github.com/nemfoundation/nem2-desktop-wallet/blob/cc56735f1ee78576856ff3d493efc9a22cfbeb71/src/components/forms/mosaic-creation/MosaicCreationTs.ts
AFAIS, these is no significant difference to me...
Am I missing something? thanks in advance.
To Reproduce
Steps to reproduce the behavior:

Download the sample code (let say JavaScript) from https://nemtech.github.io/guides/mosaic/creating-a-mosaic.html
replaced networkType, privateKey, networkGenerationHash and nodeUrl values with my own ones
run $ node CreatingAMosaic.js on terminal
the output TransactionAnnounceResponse { message: 'packet 9 was pushed to the network via /transaction' }

I cannot see any error. the output itself looks normal.
Expected behavior
A new mosaic should be created.
Then, a new mosaic entry should be added in a response of the following API:
http://<CATAPULT_SERVER_URL>/account/<OWNER_PUBLIC_KEY>/mosaics
I would appreciate if there is another way to debug my problem.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS Catalina
Version: 10.15.2

Additional context
none

Hi @gologo13, I just executed the code and seems to be working as expected.
TransactionAnnounceResponse { message: 'packet 9 was pushed to the network via /transaction' }  means that the transaction has reached the network, but not necessarily that was accepted (e.g. you might have used an incorrect network type, didn't have enough network currency units in your account, used an unexpected networkGenerationHash... ).
To debug the problem, you could add a console.log(signedTransaction.hash); just after the transaction is signed and run node CreatingAMosaic.js. Then, open the URL http://localhost:3000/transaction/<hash>/status and you see there why this transaction is not being processed.

Hi @dgarcia360 thank you for your kind response.
you're right. I got the following response from /transaction/<hash>/status:
{
hash: "5F63DB18CA4E07F43B30EABCB0657F8C42043AFEFC28B77302408863A614C63D",
code: "Failure_Core_Future_Deadline",
deadline: "118871889910",
group: "failed"
}

Looks like there's something wrong with system clocks in my environment.
sorry to bother you. I close this issue.


Not sure what I am doing

@Anclagen
I moved the folder into modules/offer now as you suggested.
I was just afraid I would mess up anything if i merged it with the dev branch so I just kept it in the middleware branch.

The reason for the dev branch was to allow for messing up a little, extra testing, and getting everything running smoothly. Anything that gets merged into the main gets deployed to the running server, we added protections to dev, like the main so the QA/Scrum Master would check the PRs into it, make sure the code was working then, either approve or request changes, and add some tests. We wanted to make it so all PRs into the main were fully tested and came with a complete passing suite of Jest tests, checking every possible variation of requests we could think of to ensure the desired response was returned.
I suggest you close this PR and open one into the dev branch it shouldn't cause too many problems, and can just be imported into the offer router.

alright :) i closed it now


ImportError: cannot import name APNs

by the usage:from apns import APNs
ImportError: cannot import name APNs

same problem here

Is the file that does the imports called apns.py too? Then add
from __future__ import absolute_import

At the top of you file.


Urgot is rolled up and doesn't display much animating

Tested in Emscripten (LeagueModel Application built on Dec 13 2018 at 22:45:11)

Same for hecarim in latest build.


feat: apollo federation 2.7

update federation version to 2.7
add directives:

authenticated
requiresScopes

note:
2.7 version supports more directives, for now only required by us are added

leave it as separate branch for now


Remove cyclic imports

Removes the import cycle of index.js importing withSafeArea which imports index.js, and so on.

Can we get this merged ? With the release of 0.57 approaching it's going to be a pain for more and more users

@gurglet  Good job, buddy!


Running unit tests changes wallet.db file committed into the repo

Running make unit modifies file cmd/chantools/testdata/wallet.db:
$ make unit
...
$ git diff
diff --git a/cmd/chantools/testdata/wallet.db b/cmd/chantools/testdata/wallet.db
index 4317346..ceb8d63 100644
Binary files a/cmd/chantools/testdata/wallet.db and b/cmd/chantools/testdata/wallet.db differ

This results in unwanted updates of the file, unrelated to a change.
I propose to change the unit test to copy a file to temp directory first and to run the modifications needed for the test on the temp file.

Yeah, I'm aware of that unfortunate side effect. I usually just reverted the change before committing, which isn't great :sweat_smile:


Fix export in "Usage" code blocks

The example configuration objects on svgo.dev assign to module.export, instead of the standard module.exports. Copying the example usage as-is causes the configuration to be silently ignored.
This commit updates module.export to module.exports.

Thanks for spotting the issue and submitting a patch!


diff cover json report don't contain the total lines' number

Is that any support that the json report contains the diff total lines' number?
The total lines number is import for report display? Is it should be given?
Json report is generated in the below cmd:
diff-cover my_jacoco.xml   --compare-branch origin/master --src-roots my_project/my_module/src/main/java  --json-report my_diff_cover_report.json
The report contains total_num_lines and violation_lines and its line number but no line number as total_lines.
{
  "report_name": "XML",
  "diff_name": "origin/master...HEAD, staged and unstaged changes",
  "src_stats": {
    "my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java": {
      "percent_covered": 50.0,
      "violation_lines": [
        13
      ],
      "violations": [
        [
          13,
          null
        ]
      ]
    }
  },
  "total_num_lines": 2,
  "total_num_violations": 1,
  "total_percent_covered": 50
}

The git diff git diff origin/master...HEAD:
diff --git a/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java b/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java
index 1633ede..52b785e 100644
--- a/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java
+++ b/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java
@@ -9,6 +9,9 @@ public class JCalculator {
         if (b == 0) {
             throw new ArithmeticException("Divide zero!");
         }
+        if (b < 0) {
+            throw new ArithmeticException("Divide neg!");
+        }
         return a / b;
     }
 }
\ No newline at end of file

The git diff git diff origin/master...HEAD:
diff --git a/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java b/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java
index 1633ede..52b785e 100644
--- a/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java
+++ b/my_module/src/main/java/com/zsmallx/example/java_calculator/JCalculator.java
@@ -9,6 +9,9 @@ public class JCalculator {
         if (b == 0) {
             throw new ArithmeticException("Divide zero!");
         }
+        if (b < 0) {
+            throw new ArithmeticException("Divide neg!");
+        }
         return a / b;
     }
 }
\ No newline at end of file

No support currently. But I would approve and release a pr that added that

No support currently. But I would approve and release a pr that added that

@ZSmallX I'm happy to create a PR for this. Can you please let me know if my understanding is right?
You want a field in the json report which contains the number of lines which are changed. So not only the number changed AND covered lines but all the number of all changed?

@ZSmallX I'm happy to create a PR for this. Can you please let me know if my understanding is right?
You want a field in the json report which contains the number of lines which are changed. So not only the number changed AND covered lines but all the number of all changed?

Yes, nice! I think that's enough to keep all the line members of all changed in JSON report, in case of big change of a huge file, which may make HTML and Markdown report ugly and unclear.

@ZSmallX
This is released. @kasium did the work im just the messenger https://pypi.org/project/diff-cover/5.3.0/


Add a case sensitive tooltip in filter by column value analysis

Context
According to https://github.com/CartoDB/design/issues/669#issuecomment-288991034 users are experiencing issues when writing a string for the filter by column value because they don't know is case sensitive.
To tackle that in the easy way we're adding a tooltip to the "input" field in "Parameters" that says. "Select or write your text. Note that this field is case sensitive."
Steps to Reproduce

Add and Filter by column value analysis
Select a string column
Write a custom text

Current Result
There is no warning that no values will be returned if the text is written in a different case
Expected result
A warning should be

pleaseeeeeeeeeee

Stale issue. Closing 👋


fix escaped antislashes in paths (windows only)

I also had to explicitly close the template file before renaming it...


Ignore the indentNextLinePattern rule when next line satisfies the increaseIndentPattern rule

This issue derives from the issue https://github.com/microsoft/vscode/issues/209804 and https://github.com/microsoft/vscode/issues/43244#issuecomment-2042078397.
Recently we have modified the indentNextLinePattern pattern so that after a braceless if statement, on pressing Enter, we indent the next line as follows (here the dots signify white spaces):
if (true)
....

Some users have mentioned in the thread of the link above, that upon writing {} they would like the line to outdent, as follows:
if (true)
{}

We need to polish this.

This looks like the ideal solution we're looking for, as long as standard intentation rules would apply (e.g. 2 space, 4 space, tab, etc.) and it would support auto indentation in the following cases:
`
// Single line brace-less
if( true )
logic( );
// Same line brace
if( true ) {
logic( );
// etc.
}
// New line brace
if( true )
{
logic( );
// etc.
}
`

Not sure I should report this here.
Recent vscode auto indent not working perfect for me .
code:
if (1)
    |

when input "{", code become:
if (1)
    {|}

It should unindent when input "{"
if (1)
{|}

Hi what is the indentation level on that line? It will not outdent if the indentation level is not the exact default indentation level because it will be assumed that the indentation is indentation.
Could you please show a video of this behavior? I can change the code if needed.

@aiday-mar

{
"javascript.format.placeOpenBraceOnNewLineForControlBlocks": true,
"javascript.format.placeOpenBraceOnNewLineForFunctions": true,
}

Version: 1.90.2 (user setup)
Commit: 5437499feb04f7a586f677b155b039bc2b3669eb
Date: 2024-06-18T22:34:26.404Z
Electron: 29.4.0
ElectronBuildId: 9728852
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Windows_NT x64 10.0.22631

Hi thank you for the quick response. You are on the latest stable release. A fix for this has been merged recently and is in the latest Insiders release. The next stable release will come out this week. The current behavior on the latest Insiders release is:
https://github.com/microsoft/vscode/assets/61460952/a6053375-f6c7-4fbc-8601-71e1768c0f17

This issue will be closed, as the fix is in Insiders


Gdscript - Ability to leave midline comments

Right now we can add a code comment on an entire line:
func _ready():
       ## this is a comment
	connect("text_changed",self,"textChangeFunc")

Or at the end of a line, like this:
func _ready(): ## this is another comment
	connect("text_changed",self,"textChangeFunc")

But it seems to be impossible to leave a comment in the middle of it.
func _ready():
	set_position(/*expects Vector2*/)
        myFunction(/*expects float*/)

This would be very useful for an addon that I am working on at the moment here:
https://github.com/godotengine/godot/issues/17795

Correct, you can not put comments in the middle of code. Just put an explanation comment above your code or right next to it. In your example the engine would not even compile because it would look for appropriate variable types even if you aren't ever using the functions. you could also put place holder variables if you aren't using them yet.
func _ready():  #comment
set_position( Vector2(0,0) ) #set_position(expects Vector2)
#myFunction(expects float)
myFunction( float(0) )

The comments being left midline can also be used as a meta-tag placeholder space for something to be replaced or generated by an addon - so I see this as a potentially powerful tool for doing interesting new ways to generate gdscript code. I will present a practical example soon hopefully.Still working on parsing gdscript to bbcode label and back. Maybe if this is not possible, I will come up with another way to do it
Not sure if other programming languages can do it - python certainly doesnt allow it

I am going to close this request since the pull request implementing it was not accepted


EmailBundle:Form/Model/Email::removeAttachment - collection::remove expects a key as arg, not value (attachment)

vendor/oro/platform/src/Oro/Bundle/EmailBundle/Form/Model/Email.php
public function removeAttachment(EmailAttachment $attachment)
{
    if ($this->attachments->contains($attachment)) {
        $this->attachments->remove($attachment);
    }
}

remove() expect a collection key but a collection value is provide in current code.

internal issue ID - BAP-17462

Hi @dgspurgin,
Thank you for contribution, feel free to create pull request with the fix.


[Menu] TypeError: key is undefined #4733

[ ] PR has tests / docs demo, and is linted.
[ ] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[ ] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Closes #4733.

@bgpat Good catch! Thanks.


Improve compilation times of pipeline apps

I have done some profiling and discovered slow compilation times are due to derivation of implicits taking a huge amount of time.
In this PR expensive implicits are cached so that they are not derived unnecessarily. There are huge improvements in compilation times for all pipeline apps:

Project
Before
After
Speed-up

internal_model
1.5 secs
15 secs
0.1 x

transformer_sierra
49 secs
1.5 secs
33 x

transformer_sierra-test
121 secs
12 secs
10 x

recorder
89 secs
0.8 secs
111 x

recorder-test
299 secs
0.6 secs
498 x

matcher
55 secs
0.6 secs
91 x

matcher-test
367 secs
3 secs
122 x

merger
111 secs
0.6 secs
185 x

merger-test
418 secs
1.5 secs
279 x

ingestor
73 secs
0.3 secs
243 x

ingestor-test
143 secs
0.6 secs
238 x

id_minter
0.6 secs
0.3 secs
2 x

id_minter-test
181 secs
0.7 secs
259 x

Note that internal_model compilation is 10 times slower in the above table, as it is here that the implicits are being derived. If this is an annoyance we can always move the implicit calculations to it's own seperate app.

👏

Very nice.


Use slugs instead of ids in the wishlist urls

Right now, our wishlist urls look like this: /wishlists/1. URLs are part of the user interface and ids don't make for a great UI; let's make them more like this: /wishlists/dc-general!
The minimum requirements for this issue are:

URLs should be slug-based instead of id-based
The slug should be automatically generated based on the name
The slug should update if the wishlist name updates
The slug (re-)generation should be fully tested
Everything should be indexed appropriately

I'd like to give this a shot!

@leesharma Does it matter if it is custom or uses a gem?

@reneesandefur Awesome! It looks like you've gotten started, but here's an answer anyway: gems are fine as long as they're good quality and there's a reason to use them. 🙂
If you add a gem, it should have decent code quality, be tested, and (preferably) be pretty common. It should also be proportional to the problem: e.g. adding a giant state machine gem to avoid a small amount of hand-written code wouldn't be worth it.
For this issue, either method (custom made or friendly_id) would work, though I'd prefer custom; friendly_id looks a bit bloated for the task at hand.


Update forgotten password-routine

This issue is imported from pivotal
If a user forgets his password, he can still login using the magic link. But to change the password, he needs to enter his old password. This makes it impossible to reset a forgotten password.

Instead of asking the old password, a logged in user should only enter his new password twice. An email is sent to the user informing him of the password change, so it can't be changed without the user knowing (eg. on an unattended device).
https://github.com/OpenConext/OpenConext-myconext/pull/59

So the scenario where the "old password" question tries to protect against is that someone has access to a logged in eduID account but not access to the mailbox of that user.
(If they would have mailbox access they do not need to change the password in the first place + the suggested notification is moot because they could just delete it.)
Why don't we just email the password reset link to the user? "Reset password" is just a button, it triggers an email, clicking on the link in the email allows you to set a new password. I believe this is a common strategy for account recovery so people will understand it. (Thijs Kinkhorst - Sep 29, 2020)

It might feel 'double' for a user to receive a password reset link via email just after he has loggen in using a link via email.
But yes, it's a widely used way for password recovery, so let's implement it this way. (Peter Havekes - Sep 29, 2020)


Add electricity symbol for low power

Instead of having a circle for low power, i'd like to see a lightning bolt looking thing. Yellow lightning bolt for low power, and red for no power.
If electricity get's it's own symbol, then I don't have to give it it's own color (like purple) to be able to tell what's wrong.

Unfortunately, this isn't possible due to game engine limitations. Sorry.


🔧 [Refactor] Delete query when redeeming the Redeem link

🔧 Refactor ticket

When redeeming the Redeem link, there is still one too many prompts.
You click on Redeem and are then asked whether you want to redeem. This query can be omitted.

Please check the following behavior as well. A transaction/contribution link is not valid. You login, errors are toasted but you are still asked to redeem the link.


Responds to alcohol posts

If a message contains an alcohol-related emoji, respond with a direct mention response and one of the following messages:

Ah, a tipple would be nice, old bean.
Yes, please!
Cheers, old sport!
Don't mind if I do, sir!
Bottoms up, old sport!

and end the message with an alcohol-related emoji.

fixed in commit 8424518 to main


Option to only get vulnerabilities from a namespace

I only need to be looking at RHEL based vulnerabilites. I'm seeing now how i can change either the datastore or the updaters to only handle RHEL based vulnerabilities. What interfaces can i change to get this option.

Recompiling clair without the imports of the components you don't want will do the trick.
https://github.com/coreos/clair/blob/4246c5244bbcffb9d278617a80bbe184bb8e935d/cmd/clair/main.go#L28-L45

thanks for the quick response!


Item dupe bugs

There are critical bugs with items, trading and storage that allow players to duplicate items and gold. Details in discord

+1

I've got reports and trying to fix it for several weeks, you don't have to repost it again, without how to reproduce it or fix it.


Forward plugin reset params in controller

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html

Original Issue: https://api.github.com/repos/zendframework/zendframework/issues/7685
User: @popovsergiy
Created On: 2016-03-13T00:50:21Z
Updated At: 2016-03-15T20:33:52Z
Body
I build interface which can render content in tabs and separately by url. I use for this forward plugin in controller and after second call params return null for all
public function indexAction() {
    $params = $this->plugin('params');

    $productsGridVm = $this->forward()->dispatch(
        $params->fromRoute('controller'), // here controller is 'store'
        ['action' => 'products']
    );
    $orderGridVm = $this->forward()->dispatch(
        $params->fromRoute('controller'), // here controller is 'null'
        ['action' => 'orders']
    );

    $view = (new ViewModel())
        ->setTemplate('tab/switcher')
        ->addChild($productsGridVm, 'products')
        ->addChild($orderGridVm, 'orders');

    return $view;
}

but if use RouteMatch object all works as expected
public function indexAction() {
    $route = $this->getEvent()->getRouteMatch();

    $productsGridVm = $this->forward()->dispatch(
        $route->getParam('controller'),
        ['action' => 'products']
    );
    $orderGridVm = $this->forward()->dispatch(
        $route->getParam('controller'),
        ['action' => 'orders']
    );

    $view = (new ViewModel())
        ->setTemplate('tab/switcher')
        ->addChild($productsGridVm, 'products')
        ->addChild($orderGridVm, 'orders');

    return $view;
}

This is messy implementation detail where plugin switches context to forwarded controller but it does not switch it back unless you access it via $this->plugin('params') again or via $this->params();
From abstract controller:
    public function getPluginManager()
    {
        if (! $this->plugins) {
            $this->setPluginManager(new PluginManager(new ServiceManager()));
        }
        $this->plugins->setController($this); // <-- this is where context switch happens
        return $this->plugins;
    }

    public function plugin($name, array $options = null)
    {
        return $this->getPluginManager()->get($name, $options);
    }

Unfortunately it is a won't fix.


click not adding modifiers properly

this pr solves issue of modifier not being appended in else case
and modifier chaining
<a v-on:click.stop.prevent="doThat"></a>

@hhaensel would you be kind enough to review this and merge :)

@hhaensel I think you missed this notification. Can you please review this PR so I can merge it?

Sorry, I really missed this one. I made slight changes to support both String/Symbol and Vector of String/Symbol syntax.

We will touch this once more, if we should agree on keyword argument syntax of the macro as described in https://github.com/GenieFramework/Stipple.jl/issues/172

We will touch this once more, if we should agree on keyword argument syntax of the macro as described in GenieFramework/Stipple.jl#172

sure

Free free to commit.
P.S: What do you think about the kw syntax?

Feel free to merge.
P.S: What do you think about the kw syntax?

I think that is really nice and solves the problem to expr going in content. I have also faced this issue previously. So does this mean we need to do changes in all macros @iif @els etc?

Exactly. I already started building such macros exemplarily and it would not be a very demanding work.
I'll give it a go then.

@hhaensel maybe you can push some and I can contribute to same branch?

I already pushed to hh-kw-macros. It's almost done. Some changes have to be pushed to StippleUI that are dependent...

Please check out hh-kw-macros in both Stipple and StippleUI. I moved @click from StippleUI to Stipple.Elements. I think it's all ready now.
Next StippleUI version needs to compat the respective Stipple version...

thanks @hhaensel


Better handling of wmic output on Windows

Description
As wmic is a Windows command, its output contains \r (quite erratically). It also appends space-padding at each line. This PR should fix some faulty parsing.
Raw wmic looks like:
$ printf '%q\n' "$(wmic os get Caption)"
$'Caption                             \r\r\nMicrosoft Windows 10 Professionnel  \r\r\n\r\r'

With according trimming, it looks like:
$ printf '%q\n' "$(wmic_query os get Caption)"
Caption\ Microsoft\ Windows\ 10\ Professionnel

get_de() suffered from that and guessed "Aero" instead of "Modern UI/Metro" on Windows 10.
Issues
Nothing from what I witnessed. Might be problematic if wmic outputs more than one line (header omitted).

I don't think this is needed a we trim white-space regardless. Thanks for the PR :+1:


Use jsPDF to open PDFs?

Hello,
I am looking at the documentation and my impression is that this library does not support opening random uploaded PDF files from the client's device, is that correct? If so, are there any plans to support this behaviour in the future?
Thank you.

Not in the near future
Duplicate of #131