TypeError: spotifyApi.createAuthorizeURL is not a function

I recently my project (that has been dormant for a while) from version 4.0.0 to version 5.0.0 of your wonderful project.
It seemed to work fine at first, but when running the frontend, it get an TypeError: spotifyApi.createAuthorizeURL is not a function.
I have initialised the library as below:
const spotifyApi = new SpotifyWebApi({
  clientId: process.env.REACT_APP_SPOTIFY_CLIENT_ID,
  redirectUri: redirectUri,
});

export default spotifyApi;

export const createAuthorizeURL = (): string => {
  const newState = generateRandomString(16);
  localStorage.setItem('authState', newState);
  return spotifyApi.createAuthorizeURL(scopes, newState);
};

I could not find anything about this specific error in the changelog, existing issues or the current documentation. Did I miss anything?
If I downgrade to 4.0.0 again it works without any other change.

Yep, also getting the same but with clientCredentialsGrant. Doesn't work on 4.0.0 or 5.0.0.

Yep, also getting the same but with clientCredentialsGrant. Doesn't work on 4.0.0 or 5.0.0.

Yeah I didn't have the feeling that it depends in the auth flow chosen.

I thought this was an issue with my environment, but I'm getting this as well in 5.0.0 when trying to use the SpotifyWebApi in a Nuxt application
client.js?06a0:97 TypeError: spotify.createAuthorizeURL is not a function
    at eval (spotify.client.js?1450:14)
    at _callee2$ (index.js?f26e:64)
    at tryCatch (runtime.js?96cf:63)
    at Generator.invoke [as _invoke] (runtime.js?96cf:293)
    at Generator.eval [as next] (runtime.js?96cf:118)
    at asyncGeneratorStep (asyncToGenerator.js?1da1:3)
    at _next (asyncToGenerator.js?1da1:25)

I thought this was an issue with my environment, but I'm getting this as well in 5.0.0 when trying to use the SpotifyWebApi in a Nuxt application
client.js?06a0:97 TypeError: spotify.createAuthorizeURL is not a function
    at eval (spotify.client.js?1450:14)
    at _callee2$ (index.js?f26e:64)
    at tryCatch (runtime.js?96cf:63)
    at Generator.invoke [as _invoke] (runtime.js?96cf:293)
    at Generator.eval [as next] (runtime.js?96cf:118)
    at asyncGeneratorStep (asyncToGenerator.js?1da1:3)
    at _next (asyncToGenerator.js?1da1:25)

spotifyApi.createAuthorizeURL(scopes, state) does work on 4.0.0 but it doesn't in 5.0.2... I guess a temporary workaround is downgrading until the issue is resolved.

spotifyApi.createAuthorizeURL(scopes, state) does work on 4.0.0 but it doesn't in 5.0.2... I guess a temporary workaround is downgrading until the issue is resolved.

Looks like the server methods are missing.
This is my 1am fix if you don't want to downgrade:
npm i spotify-web-api-node
npm i @types/spotify-web-api-node --save-dev

import SpotifyWebApi from 'spotify-web-api-node';
import SpotifyWebApiServer from 'spotify-web-api-node/src/server-methods';

// (...)

// add server methods to client methods
(SpotifyWebApi as unknown as { _addMethods: (fncs: unknown) => void })._addMethods(SpotifyWebApiServer);
this._api = new SpotifyWebApi();
this._api.createAuthorizeURL()

(taken from source: https://github.com/thelinmichael/spotify-web-api-node/blob/2785f79d58a9017e9f57fbb0f06e6d23022fd3b4/src/server.js)
Obviously this is an unofficial fix and may break in the future. (works for 5.0.2)

Looks like the server methods are missing.
This is my 1am fix if you don't want to downgrade:
npm i spotify-web-api-node
npm i @types/spotify-web-api-node --save-dev

import SpotifyWebApi from 'spotify-web-api-node';
import SpotifyWebApiServer from 'spotify-web-api-node/src/server-methods';

// (...)

// add server methods to client methods
(SpotifyWebApi as unknown as { _addMethods: (fncs: unknown) => void })._addMethods(SpotifyWebApiServer);
this._api = new SpotifyWebApi();
this._api.createAuthorizeURL()

(taken from source: https://github.com/thelinmichael/spotify-web-api-node/blob/2785f79d58a9017e9f57fbb0f06e6d23022fd3b4/src/server.js)
Obviously this is an unofficial fix and may break in the future. (works for 5.0.2)

Thanks for sharing this code.
Can you tell me how can i add this in reactjs as it is js not Typescript?

This is still an issue when I tried to import and use as follows:
Version: ^5.0.2
Svelte-Kit App
(works on server side, not on client side).
import SpotifyWebApi from 'spotify-web-api-node';

export function getAuthUrl() {
   // ...

	const spotifyApi = new SpotifyWebApi({
		redirectUri: redirectUri,
		clientId: clientId
	});

        // Uncaught (in promise) TypeError: spotifyApi.createAuthorizeURL is not a function
	const authorizeURL = spotifyApi.createAuthorizeURL(scopes, state, showDialog);

	return authorizeURL;
}

Solution based on @ssubham's above works.
import SpotifyWebApi from 'spotify-web-api-node';
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore
import SpotifyWebApiServer from 'spotify-web-api-node/src/server-methods';

(SpotifyWebApi as unknown as { _addMethods: (fncs: unknown) => void })._addMethods(
	SpotifyWebApiServer
);

Thanks for sharing this code. Can you tell me how can i add this in reactjs as it is js not Typescript?

The following code worked for me with JS.
import  SpotifyWebApi from 'spotify-web-api-node';
import  SpotifyWebApiServer from 'spotify-web-api-node/src/server-methods';

SpotifyWebApi._addMethods = function(fncs) {
  Object.assign(this.prototype, fncs);
};
SpotifyWebApi._addMethods(SpotifyWebApiServer);


python312Packages.tencentcloud-sdk-python: 3.0.1166 -> 3.0.1167

Diff: https://github.com/TencentCloud/tencentcloud-sdk-python/compare/refs/tags/3.0.1166...3.0.1167
Changelog: https://github.com/TencentCloud/tencentcloud-sdk-python/blob/3.0.1167/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 319483 run on x86_64-linux 1

4 packages built:

python311Packages.tencentcloud-sdk-python
python311Packages.tencentcloud-sdk-python.dist
python312Packages.tencentcloud-sdk-python
python312Packages.tencentcloud-sdk-python.dist


Sierra Dynamo Sink

What is this PR trying to achieve?
Add akka streams sink to insert a sierra record to dynamo db
Who is this change for?
Devs
Have the following been considered/are they needed?

(https://wellcomecollection.org/works/hkg8pw2k?query=sink)


Add Thomas Bohnert to ARFC Website

Summary of changes

Added image to img/people and bio to file people.yml
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Associated Issues and PRs

Issue: #

Associated Developers

Dev: @

Checklist for Reviewers
Reviewers should use this link to get to the
Review Checklist before they begin their review.

Sorry for being so late on this @ThomasBohnert. Looks like you haven't checked off any of the self-review boxes. Have you tested that the site builds with your changes?


rules_cuda_dependencies() & register_detected_cuda_toolchains()  WILL REMOVE @platforms//os:visionos'\

I used rules_cuda v0.2.1 to import GPU support in my bazel projects.
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
http_archive(
    name = "rules_cuda",
    sha256 = "9b8758a82b57b7d06ebc2449753f8ac9a925f5ef3fa6a84882263c47e697d929",
    strip_prefix = "rules_cuda-0.2.1",
    urls = ["https://github.com/bazel-contrib/rules_cuda/archive/0.2.1.tar.gz"],
)
load("@rules_cuda//cuda:repositories.bzl", "register_detected_cuda_toolchains", "rules_cuda_dependencies")
rules_cuda_dependencies()
register_detected_cuda_toolchains()

It worked well when I build a cuda library, but when I tried add some test, I got "no such target '@platforms//os:visionos'".  I used  bazel query "@platforms//os:*" and find
@platforms//os:BUILD
@platforms//os:android
@platforms//os:freebsd
@platforms//os:fuchsia
@platforms//os:ios
@platforms//os:linux
@platforms//os:macos
@platforms//os:netbsd
@platforms//os:nixos
@platforms//os:none
@platforms//os:openbsd
@platforms//os:os
@platforms//os:osx
@platforms//os:qnx
@platforms//os:srcs
@platforms//os:tvos
@platforms//os:wasi
@platforms//os:watchos
@platforms//os:windows

There are no  @platforms//os:visionos , but when I remove the two function
rules_cuda_dependencies()
register_detected_cuda_toolchains()

I found that
@platforms//os:BUILD
@platforms//os:android
@platforms//os:chromiumos
@platforms//os:freebsd
@platforms//os:fuchsia
@platforms//os:haiku
@platforms//os:ios
@platforms//os:linux
@platforms//os:macos
@platforms//os:netbsd
@platforms//os:nixos
@platforms//os:none
@platforms//os:openbsd
@platforms//os:os
@platforms//os:osx
@platforms//os:qnx
@platforms//os:srcs
@platforms//os:tvos
@platforms//os:visionos
@platforms//os:wasi
@platforms//os:watchos
@platforms//os:windows

I got several items which seems to be removed by rules_cuda_dependencies() & register_detected_cuda_toolchains(), leading to error when testing. I did not know why this happen, and how can I avoid this error ?

You need to load the platforms repo in your WORKSPACE file before calling the rules_cuda macro.

@Autumn1998 please let me know if my workaround doesn't work for this and if it is still an issue and we can reopen this.


Fix inconsistency of medals having emtpy ids in packs

because there were no ids between these maps, it was treating them as id 0 and if 0 in the db is just 1, so the response of this request was [actualcount,1,1,1], when in reality it should just be [actualcount].

sorry, but please stop editing the database like this. this is impossible for us to update on the site, the sql is just there to get a dev environment working.


Adds support for checking if a value was explicitly provided

The use-case for this is handling situations where a nil value was provided on purpose (i.e. nil isn't the absence of something, but an explicit state).
class FooSchema < Erlen::Schema::Base
  attribute :foo, Integer
end

FooSchema.import({}).attribute_provided?(:foo) # => false
FooSchema.import({ foo: nil }).attribute_provided?(:foo) # => true

FooSchema.new({}).attribute_provided?(:foo) # => false
FooSchema.new({ foo: nil }).attribute_provided?(:foo) # => true

Coverage increased (+0.02%) to 97.959% when pulling b8b6504a13894b59e7b5104d3faf2da014c2b438 on arlandism:has-attribute into a3ef65534e5f14d47acf620a66b4377fa0d25ac9 on Hireology:master.

@timpbrenner Should I bump the version in this PR?

@arlandism Yes, bump the version here.


Corrected the GitHub link to point to the nginx.json file

Corrected the GitHub link to point to the nginx.json file - https://raw.githubusercontent.com/kubernetes/ingress-nginx/main/deploy/grafana/dashboards/nginx.json

@sabbour
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Clarify the distributed functions

I realized that several users are confused by create_distributed_function and its purpose. I think the main confusion comes from this: create_distributed_table makes things faster, so let's try create_distributed_function to make functions faster. Also, they refer to this blog and probably skimming through it and conclude it makes things faster.
It'd be really nice if we can improve the documentation on this topic so that users do not try to distribute functions unnecessarily / with wrong expectations.
(below copy & paste from an internal e-mail)
What are the typical use cases?
There 3 two main use cases for distributed functions, and as far as I can tell, the function you try to distribute does not fall into any of the categories.

The primary use case for distributed functions with distribution_arg_name and colocate_with parameters:

It is mostly useful if you need to scale OLTP workloads that are written as stored procedures.
Say, you have the following stored procedure where all the commands are scoped to per tenant/distribution key:
CREATE FUNCTION procedure_to_pushdown (int dist_key, …)
BEGIN;
      INSERT INTO dist_table (dist_key, …) .. VALUES ($1_;
     UPDATE dist_table … WHERE dist_key = $1;
    DELETE FROM collocated_dist_table WHERE WHERE dist_key = $1;
…
COMMIT;
end function;

If you do not create distributed functions, each statement will become 1 round-trip to the worker node from the coordinator.
Instead, if you convert this function to a distributed function by providing relevant parameters, the procedure call will be routed to the correct node (e.g., we can deduct it from the distribution parameter of the function).
SELECT create_distributed_function(' procedure_to_pushdown (int)', ‘dist_key’, collocate_with:‘distributed_table’);
With that, Citus needs only 1 round-trip to the worker node per procedure call. This might save a lot of round-trips, especially if the function has lots of commands in it.
And then, you get all the benefits described in this blog

Secondary use-case for distributed function:

If you have very simple functions that DO NOT return rows, but can only be used in the SELECT clause. Say you have this value_plus_one() function. If you use this function on a distributed query, it’ll use this function on the shards and you’ll get an error (with some logs enabled):
SELECT value_plus_one(key) FROM distributed_table;
….
NOTICE:  issuing SELECT public.value_plus_one(a) AS value_plus_one FROM public.test_102012 test WHERE true
ERROR:  function public.value_plus_one(integer) does not exist
HINT:  No function matches the given name and argument types. You might need to add explicit type casts.
CONTEXT:  while executing command on localhost:9705

As a convenience to be able to create the functions on the workers, you can use create_distributed_function without distribution arguments:
SELECT create_distributed_function('value_plus_one(int)');

With that, the function will be available on the worker, and the error can be avoided:
select value_plus_one(a) from test;
value_plus_one
----------------
(0 rows)

Time: 127.368 ms

In other words, this is mostly equivalent for running CREATE FUNCTION .. commands on the workers, no magic happens.

Load balancing function calls that are co-located with reference tables

/* todo: fill up later */

Maybe we should rename the function to create_colocated_function or such


Is rocprofv2 part of ROCm?

just like rocprof v1?

rocprofv2 is more of an alpha/beta at this point. The command line tool is really just a vehicle for testing the new rocprofiler v2 API implementation.


UIA Operation Abstraction lib: implement support for CallExtension / IsExtensionSupported

UI Automation recently introduced the concept of custom patterns. This is the ability for a provider to expose custom methods, identified by a GUID, that a client an execute via UIA remote Operations. For example, MS Word might expose a custom pattern for moving by sentence - something that UIA by itself does not support.
This PR implements the ability for a client to execute these custom methods (extensions) by using the UIA Operation Abstraction library. Support for custom patterns has already been introduced to the Microsoft::UI::UIAutomation winrt library.
Specific changes:

Added a new UiaGuid class, which represents either a local or remote GUID.

It can be constructed locally from a winrt::guid.
It can be assigned to, and compaired with == / !=.
It exposes LookupAnnotationType and LookupPropertyId methods, which work both remotely and locally.

Implement UiaElement::IsExtensionSupported: this method takes a UiaGuid as an argument, and returns a UiaBool denoting if the extension identified by the given GUID is supported by the remote provider.

If remote, this method calls AutomationRemoteElement::IsExtensionSupported.
This method has no local equivilant and therefore throws NotImplemented if executed locally.

Implemented UiaElement::CallExtension: this method takes a UiaGuid, plus a variable number of further arguments of any Uia* type, and executes the extension identified by the given GUID, passing the given arguments as its in/out arguments.

If remote, this method calls AutomationRemoteElement::CallExtension.
This method has no local equivilant and therefore throws NotImplemented if executed locally.

No tests have been written yet. These will be hard to write unless they can be run against a specific app that exposes 1 or more custom patterns.
Manually I have tested UiaGuid::LookupAnnotationType / UiaGuid::LookupPropertyId. But I am so far unable to test UiaElement::IsExtensionSupported / UIAElement::CallExtension until I have an app that actually implements 1 or more custom patterns...

I have been able to manually confirm that these changes work, at least for my use cases, by testing them against Microsoft Word's custom patterns implementation, from within the NVDA screen reader. However, I really am not sure how this can be tested specifically within this project as access to Microsoft Word, or something that implemented custom patterns, would be required on the build system.

Sorry for us ignoring this PR for so long! I'll give it a look next week. @mavangur @zhayuli @pengyuanjin for FYI.
Honestly, we haven't even had the time to get our tests running in CI yet. There's a general issue with launching apps and we haven't really gotten to look at why. I'll give some thought as to what we can do with regard to automated testing here.

I tried to add support for GUIds to UIAVariant, but did not succeed.  I
am unsure as to what vt type should be used, or if it it is at all
possible to hold a GUID in a VARIANT structure at all these days?
I'm happy to either remove these commented lines, or if anyone has
further info on how to do this properly, please share and I'll try
adding that support again.
Message ID:
@.***>

@michaelDCurran I'd say as long as we don't need GUIDs in UiaVariant for this PR, we can remove those comment lines. It might make sense to store GUIDs in VT_SAFEARRAY in the variant, though, when we want to implement that.
Also, we could add support for extension tests by creating a provider that supports custom extensions, and loading it with the ModernApp test utility (or creating a Win32 executable). If you feel you're up to that in this PR feel free (and of course ask us if you need help!), but if not I've created #86 to track adding those tests in the future. I can try to get around to confirming your manual testing next week.

I'm really sorry for all the slow responses here!
My remaining comment is can we add tests for UiaGuid? We don't currently have a huge amount of tests for each individual type, but I figure we should aim for good test coverage on new types. So basically calling all its methods, operating locally and remotely, etc.
Otherwise, this looks great!!!

@michaelDCurran just in case you missed these notifications.

Sorry for delay on this. Hopefully I should find time in next week or so
to write UiaGuid tests.

No need to apologize I just wanted to make sure you didn't think you were waiting on me :)

@beweedon  I'm afraid that I can't think of any ways I can write remote tests for UiaGuid.

There is no method (other than UIA custom extensions) that take a remote guid
There are no UIA methods that give back a guid
A guid to / from a custom property or annotation only makes sense if there are custom annotations or properties already registered in the remote provider.

In other words, to create any meaningful tests for UiaGuid, (just like for custom extensions) we'd need to write a custom provider or test with an MS Office product such as Microsoft Word.
I don't believe that I am qualified at the moment to pull that one off at this stage. Plus, some of the other additions we have talked about are probably more high priority for me. Thankfully those ones are actually quite easy to write tests for ;)
I'll leave it up to you then as to if you wish to merge this with no custom provider / tests. I'm already using this code real-world in NVDA 2022.1 via my fork michaeldcurran/microsoft-ui-uiautomation with the code from this pr merged.

@michaelDCurran I was thinking mostly about testing the simple stuff just to make sure it works.

Test that UiaGuid can go from local to remote back to local again.
Test that UiaGuid assignment and equality work.
Test that LookupPropertyId works if you set the UiaGuid to one of the known property GUIDs in UIAutomationCoreApi.h (e.g. Name_Property_GUID should correspond to UIA_NamePropertyId).

I'm glad you're not blocked, though :). If you don't want to add the tests, one of us can look into that at some point and get this merged in.

@beweedon  I've now added tests for UiaGuid, including equality checks, and looking up property IDs and annotation type IDs.

@michaelDCurran This looks great to me! I'm going to see if other reviews filter in before merging.


update eslint

chore: update eslint config; add eslint-plugin-perfectionist
chore!: fix engines
BREAKING CHANGE: Supported Node.js versions are now ^20.13.0 || ^22.0.0
chore: make commitlintrc a module
chore: lint

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#222  👈 (View in Graphite)
#221
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


test: added unit tests for react-dnd-html5-backend

Added unit tests for react-dnd-html5-backend files.
There are some private methods in class/functions inside functions which cannot be directly tested. I have not made any code refactor in source files to make it easily testable. This PR is to add the test cases for code which can be easily tested.

Thank you @sant0shg !


Clarify role handling

I just learned about this, which sounds like a great addition to Azure managed Postgres. I read through this, but maybe I missed it, so here are some points I found missing/confusing:

(cosmetic) the text says DMS is available from version 10, but one of the screenshots mention a 9.6 instance as source.

It's unclear how database users/groups/roles are migrated; that is not part of the pg_dump , but it does not seem to be part of the later DMS-specific steps either, or if it is, it's not mentioned and should be clarified.

Somewhat connected to the second point, it's not mentioned which kind of privileges the role(s) used for the logical replication need to have. Is it being part of the azure_pg_admin group? Is it being the owner of the database to be migrated?

AFAIK, no DDL (like CREATE TABLE, or changing table columns) are allowed during the time DMS is running, but this does not seem to be mentioned. There's a few other limitations of logical replication that could be mentioned as well (no TRUNCATE, no partitioned tables until v13, etc.)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 00403cac-9788-3966-2c23-696748b9379b
Version Independent ID: db980aa3-68e9-98d0-724c-030ab3fd68ff
Content: Tutorial: Migrate Azure DB for PostgreSQL to Azure DB for PostgreSQL online via the Azure portal - Azure Database Migration Service
Content Source: articles/dms/tutorial-azure-postgresql-to-azure-postgresql-online-portal.md
Service: dms
GitHub Login: @arunkumarthiags
Microsoft Alias: arthiaga

@mbanck Thank you for the feedback. We are actively investigating and will get back to you soon.

@mbanck This is being assigned to the content author to evaluate and update as appropriate.

@arunkumarthiags Arun, hello, let me know if you can take a look at this issue and provide your feedback. Thank you!

We have also come across this role issue as pg_dump only takes schema backup and we have databases configured with custom roles.

This question is more than 18 months old, with no activity, so I am closing it. If you feel that your question still should be answered, please either open a Support request, or use the Feedback option on the article in question. GitHub issues for azure-docs will be disabled in the near future.
#please-close


Eureka server Unavailable replicas on swarm

This is a similar issue but the solution on this thread doesn't work for me. There is only one eureka instance per replica and other zones from that replica are not reachable.
Here is my sample project
I'm using swarm with two nodes on separate machines.
application.yml:
server:
  port: 8761

eureka:
  instance:
    hostname: ${HOSTNAME}
    appname: discovery
    prefer-ip-address: true
  client:
    register-with-eureka: true
    fetch-registry: true

spring:
  cloud:
    inetutils:
      ignoredInterfaces:
      - docker
      - eth.*
      - lo
      preferredNetworks:
      - 10.0

---
spring:
  profiles: peer1
eureka:
  client:
    service-url:
      defaultZone: http://${HOSTNAME}:8761/eureka/,http://${HOSTNAME}:8762/eureka/,http://${HOSTNAME}:8763/eureka/

---
spring:
  profiles: peer2
eureka:
  client:
    service-url:
      defaultZone: http://${HOSTNAME}:8761/eureka/,http://${HOSTNAME}:8762/eureka/,http://${HOSTNAME}:8763/eureka/

---
spring:
  profiles: peer3
eureka:
  client:
    service-url:
      defaultZone: http://${HOSTNAME}:8761/eureka/,http://${HOSTNAME}:8762/eureka/,http://${HOSTNAME}:8763/eureka/

docker-compose.yml:
version: '3'
services:
  eureka1:
    image: <IP_ADDRESS>:5000/discovery:1.0.0
    ports:
    - "8761:8761"
    environment:
    - SPRING_PROFILES_ACTIVE=discovery1

  eureka2:
    image: <IP_ADDRESS>:5000/discovery:1.0.0
    ports:
    - "8762:8761"
    environment:
    - SPRING_PROFILES_ACTIVE=discovery2

  eureka3:
    image: <IP_ADDRESS>:5000/discovery:1.0.0
    ports:
    - "8763:8761"
    environment:
    - SPRING_PROFILES_ACTIVE=discovery3

networks:
  springcloud-overlay:
    external:
      name: springcloud-overlay

deploy on swarm:
docker network create -d overlay springcloud-overlay
docker stack deploy -c docker-compose.yml discovery

Result:

I think I should change defaultZone to something like this but I don't know how to detect their IP:
defaultZone:
   http://<IP_ADDRESS>:8761, http://<IP_ADDRESS>:8761, http://<IP_ADDRESS>:8761

(assume that <IP_ADDRESS> is for the first instance, <IP_ADDRESS> for the second ...)
Note:
I also disabled the firewall using ufw disable command.

@SaeedMasoumi
When you have more than 2 Eureka instances configuration of appliction.yml should be something similar to below. Try changing your application.yml with below settings.
eureka:
  client:
    serviceUrl:
      defaultZone: http://${eureka1_HOSTNAME}:8761/eureka/,http://${eureka2_HOSTNAME}:8762/eureka/,http://${eureka3_HOSTNAME}:8763/eureka/

---
spring:
  profiles: eureka1
eureka:
  instance:
    hostname: eureka1_HOSTNAME

---
spring:
  profiles: eureka2
eureka:
  instance:
    hostname: eureka2_HOSTNAME

---
spring:
  profiles: eureka3
eureka:
  instance:
    hostname: eureka3_HOSTNAME

These solutions didn't work for me unless I add 3 network aliases in each docker service and reference them in my defaultZone to find eureka servers
docker-compose.yml
version: '3'
services:
  eureka1:
    image: <IP_ADDRESS>:5000/discovery:1.0.0
    ports:
    - "8761:8761"
    environment:
    - SPRING_PROFILES_ACTIVE=discovery1
    network:
      springcloud-overlay:
        - eureka1
        - eureka2
        - eureka3

  eureka2:
    image: <IP_ADDRESS>:5000/discovery:1.0.0
    ports:
    - "8762:8761"
    environment:
    - SPRING_PROFILES_ACTIVE=discovery2
    network:
      springcloud-overlay:
        - eureka1
        - eureka2
        - eureka3

  eureka3:
    image: <IP_ADDRESS>:5000/discovery:1.0.0
    ports:
    - "8763:8761"
    environment:
    - SPRING_PROFILES_ACTIVE=discovery3
    network:
      springcloud-overlay:
        - eureka1
        - eureka2
        - eureka3

networks:
  springcloud-overlay:
    external:
      name: springcloud-overlay

application.yml
eureka:
  client:
    serviceUrl:
      defaultZone: http://eureka1:8761/eureka/,http://eureka2:8762/eureka/,http://eureka3:8763/eureka/

It works fine but I found some issue with Zuul, So finally I decided to remove Eureka and use docker itself as a discovery service.

In the end, have you a sample project that work fine? I'm facing the same issue and tried having aliases and reference each eureka by its hostname but still having the unavailable replicas...


Separate / allow for networking only

user story
As a nostr app dev who has strong preferences on app specific choices, I would love for nostr SDK to handle networking only, so that I can focus on app specific changes without having to worry about networking and compatibility with other clients & relays.
acceptance criteria

Nostr SDK implements best practice (e.g. outbox model) network connectivity to relays

performance requirements for Damus
@jb55 to provide
builds on
Add outbox to NDK https://github.com/nostr-sdk/nostr-sdk-ios/issues/146

On Mon, Apr 15, 2024 at 09:46:24AM GMT, alltheseas wrote:

user story
As a nostr app dev who has strong preferences on app specific choices, I would love for nostr SDK to handle networking only, so that I can focus on app specific changes without having to worry about networking and compatibility with other clients & relays.
acceptance criteria

Nostr SDK implements best practice (e.g. outbox model) network connectivity to relays

performance requirements for Damus
@jb55 to provide

yes I think having networking cleanly separated is pretty key.
I want to just hand nostr-sdk a filter and have it know what relays to
connect to. The more standardization around how notes are fetched, the
better compatibility between clients when it comes to fetching notes.
Obviously this is quite hard, as the SDK must know about the context in
which it is pulling the note, and the relay list associated with the
outbox request. This is another reason I have yet to implement it, since
it is not a simple interface.
In my use case, I really only want the raw data back, unparsed. I admit
this is probably out of scope for the sdk as its a unique nostrdb
requirement, since nostrdb parses and validates notes very efficiently.
I even going as far to only partially parsing the JSON if we already have
the note, which can be a big deal on large notes like contact lists.
If this is too much of a requirement, I'm happy just using nostr-sdk for
bits of code that I need, such as nip44, etc.


Add WDC package

I need help 😃
https://travis-ci.org/ruslo/hunter/jobs/258457421#L1172

Thanks, build is success.

Ok.

I tried remove dependencies, but it is failed.
https://travis-ci.org/rusdevops/hunter/jobs/258286597#L6682-L6696

My CMakeLists.txt:
https://github.com/CloudPolis/webdav-client-cpp/blob/v1.0.8/CMakeLists.txt#L54-L68

 Target "example" links to target "OpenSSL::SSL" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?

find_dependency/find_package should be called in *Config.cmake file.

for example: https://github.com/hunter-packages/cpp_redis/blob/hunter/cmake/Config.cmake.in ?

Yes, kind of. Also find_dependency will not respect CONFIG here as far as I remember, so I recommend to use find_package(... CONFIG REQUIRED) instead.

Example:

https://github.com/hunter-packages/ogles_gpgpu/blob/826688323a6a84e456a651488f59cf8374fa547f/cmake/Config.cmake.in#L3-L28

I think this variant would be successful.

https://travis-ci.org/ruslo/hunter/builds/258615097
https://travis-ci.org/rusdevops/hunter/builds/258611241

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2074
https://travis-ci.org/ingenue/hunter/builds/259143769?utm_source=github_status&utm_medium=notification

Some toolchains failed. Feel free to exclude them from testing, branch pkg.wdc.

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2111
https://travis-ci.org/ingenue/hunter/builds/261261227?utm_source=github_status&utm_medium=notification

Good news, tests is successful.

Uploading:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2112
https://travis-ci.org/ingenue/hunter/builds/261273646?utm_source=github_status&utm_medium=notification

Released: https://github.com/ruslo/hunter/releases/tag/v0.19.59


Fix SinkBuffer block problem

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #5222
Problem Summary(Required) ：

When pipeline_sink_brpc_dop is set to 1, considering the following codes.

_request_seqs[instance_id.lo] is 0, which is init value
_max_continuous_acked_seqs[instance_id.lo] is -1, which is init value
so discontinuous_acked_window_size is 0 - (-1) = 1
too_much_brpc_process turns out to be true even we didn't send any packet, then block occurred

        if (_is_dest_merge) {
            // discontinuous_acked_window_size means that we are not received all the ack
            // with sequence from _max_continuous_acked_seqs[x] to _request_seqs[x]
            // Limit the size of the window to avoid buffering too much out-of-order data at the receiving side
            int64_t discontinuous_acked_window_size =
                    _request_seqs[instance_id.lo] - _max_continuous_acked_seqs[instance_id.lo];
            too_much_brpc_process = discontinuous_acked_window_size >= config::pipeline_sink_brpc_dop;
        }

run starrocks_be_unittest

run starrocks_clang-format

@Mergifyio backport branch-2.2


Feature Request | Jira Priority

I would like the severity of the code scanning alert (Error, Warning, Info) to map to a priority in the Jira bug that is opened.
Error = High
Warning = Medium
Info = Low
This would make it easier for product owners, scrum masters, etc to be able to prioritize the bugs they see in their backlog at a glance. This would also prevent having to look at the code scanning alerts, and match them up with the bugs. Or have to click the individual links inside the bugs to be brought to the specific alert, to then look at the severity that it has.

The code scanning team is coming up with changes that will make the severities easier to understand. I'll keep you posted in Slack 👍🏼

On top of this Secret's should also be labeled as the Highest priority.

Added it to our organization - https://github.com/imaware/ghas-jira-integration


apinger loss_interval array key name

Thanks @phil-davis  for catching all these issues introduced with dpinger!!

It's always fun to do code review.


Compile to Wasm

Resolves #25.

Too many merge conflicts; gonna just open a new PR.

Continued in #112.


Azure Functions supporting JDK 11

I would really like to know if there is already a support for JDK 11. The issues regarding this do not have any update or follow through which is unfortunate.
Plugin name and version
azure-functions-maven-plugin:1.3.2
Plugin configuration in your pom.xml
1.3.2
Actual behavior

Steps to reproduce the problem

Add azure-functions-kotlin-archeype:1.23 to IntelliJ IDEA
Generate an Azure Functions archetype
Add the corresponding trigger function
Run function host locally
Trigger functions

@muneeba-mughal Thanks for your support, Java 11 function is not supported yet, but we have planed this item and will update here once there are updates.

@muneeba-mughal Thanks for your support, Java 11 Function has been supported now, you may get the release notes here; besides, please update the function-core-tools to the latest version for running function locally, you may find the guidance here


thai language support

Please add support for thai; dying light - hellraid (DLC) has two files in that language which are currently not found.

Added support for thai in 5c64fc3


Nullability issues running Kotlin version of Cloud Document Text Recognition

Running the Kotlin version of Cloud Document Text Recognition loading an image from the file system I spotted this issue:
detectInVisionImage of VisionProcessorBase is always called with null originalCameraImage and null metadata
The metadata check for nullability is always failing and is not executing the onSuccess even though we don't really need it in the onSuccess body.
The application is ok using the fix proposed, but proably we needed a better refactor to solve it.

Assigning to @gkaldev ... the code looks fine in isolation but not familiar with the larger issue it's addressing.

@gkaldev Any news about this?

I apologize for the massive delay on this--I somehow just muted this thread. Not sure if something changed in the SDK since this PR was initially created, but I tested just now and it seems to work as it is. I successfully used a file system image to run cloud doc text rec.


Apply the changes that fix the Marlin verifier to feat/testnet2-dpc

Motivation
This PR mirrors #222, but it is pushed to feat/testnet2-dpc.
GitHub and my local merge via git apply -3 seem to have some conflicts, so I merge manually.
Test Plan
Tests in polycommit and marlin all pass.
Related PRs
This is essentially the same PR as #222, albeit solving the merge conflicts.
It closes #210.

A few checks are failing, but it seems to be formatting changes that are not part of this PR. So, the PR is ready for review & merge.


development environment setup guide of spika android

Hello,
I am looking for a set up guide for development environment of spika android to local machine.
Can you help me in this?
Best Regards,
Ashish Kalra

http://www.spikaapp.com/en/build/android


Revisit the values for the extensible enum FhirVersion

Decide on a more friendly name for FhriVersion.Four01 :)

It's also in the wrong namespace. There's no .Models namespace.

In JS, I am using the full version as the key.

That's not a valid identifier in .NET. Like our ServiceVersions, we could maybe do "V4_0_1". What do others think? /cc @kristapratico

We just exposed a string in Python. Note that this was removed in the GA swagger, but we should revisit this issue in the post-GA beta when FHIR is re-introduced.

For the record, the type of the parameter itself is also a string: https://github.com/Azure/azure-sdk-for-js/blob/e0451efa327c944d3536e528d40ba50e2d90c0a5/sdk/textanalytics/ai-text-analytics/src/generated/models/index.ts#L1535-L1542
The type I linked to in my previous comment is the enum of known values that customers could use to discover some known values.

In cases when we expect a lot of use, we typically use a "*WellKnownValues" static class of public constant values, which is idiomatic to .NET if an enum isn't appropriate.

I've done similar to JS but defined a WellKnownFhirVersions static class, since "WellKnown" is a common prefix for usage like this. The value itself is defined as V4_0_1 which matches the pattern for our client options' ServiceVersion.

Punting to next version. FhirVersion was actually removed from HealthcareTaskParameters in 2022-05-01.

Pushing out since this isn't added back in till the next version of the service will we'll use after we GA TA.


HealthChecker func when it returns an error, the statusCondition message doesn't include the error

I'm using a healthchecker function, and when it returns an error, we get something like this in the status condition message:
Probe addon unavailable with err <nil>

Where we would expect to see the text from the error instead.

I've created a proposed fix, please let me know if you're ok with it!

Thanks @tesshuflower . merged.

/close


Club data uses db in prod

Added a check to see if in prod or dev. If in dev uses localhost:8000/cache/club-info.json, while in prod uses publish db.

Thanks

Could some one merge this as it is needed for club data to work in prod right now


Set up Dependabot groups

Current dependencies on/for this PR:

main

PR #308   👈

This comment was auto-generated by Graphite.


fix(slide-toggle): report change to model before firing a change event

Whenever the value of the slide-toggle changes through interaction, a change event will be emitted. The value change then will be reported to the ControlValueAccessor which fires the ngModelChange output.
Right now the ngModelChange output fires after the normal change output. This should be the other way around because developers expect the two-way data binding to already reflect the change.
Fixes #7074

Also fixes #7300


Resizing not working properly

What you were trying to do
Resize window
What actually happened
window resized but not the "rendering area" (something like #2490 but for blocks)
Skybox and water are rendered as expected
outline is visible on line between rendered zone and empty one (when outline is enabled in options)
How to reproduce

Step 1 turn on outilne in graphic settings
Step 2 resize window before joining server or opening local game (after loading world it won't work)
Step 3 see weird things going on

###More sophisticated explanation
After starting Terasology, entering game and resizing this bug occurs. After quitting game without quitting Terasology, entering game again and resizing bug #2490 occurs but not this one, and bug 2490 stays on the screen even after quitting game and resizing window again (skybox rendering width stays the same all the time when bug #2490 occurs)
Log details and game version
version 1 836 alpha 2016-11-05T03:55:30Z
Terasology-.log.txt
Terasology-init.log.txt
Terasology-menu.log.txt
Terasology-mmo.to-<IP_ADDRESS>-25778.log.txt
screens
http://i.imgur.com/2KSFJex.png
http://i.imgur.com/NhzE3DE.png
Computer details
OS: Ubuntu 16.10 yakkety
Kernel: x86_64 Linux 4.8.0-26-generic
Resolution: 1366x768
CPU: Intel Core i5-5200U CPU @ 2.7GHz
GPU: GeForce 920M (Nvidia drivers 370.28)
RAM: 7886MiB
Java version: 1.8.0_111

Thank you @Halamix2 for reporting this. I managed to reproduce it. I'll see what I can do about it.

While I did manage to reproduce it, it took me a while to reproduce it reliably. For posterity, the steps I found most reliable are:

Start Terasology in Windowed Mode with Outline OFF
Go to the Video Settings and turn it ON
Go to the Main Menu
Resize the window to nearly full screen
Load  up a game (everything will be normal)
Bring up the In-game menu
Resize the window back down
Exit the game (but not Terasology)
Go to the Video Settings and turn Outline OFF
Go to the Main Menu
Resize the window to nearly full screen
Load up a game: the sky is cropped

This should be fixed in #2593. Can you verify @Halamix2? Or do you need to wait for the change to be merged before you can test it?

Hey @Halamix2 could you confirm with the latest build or the develop branch and reopen this if not fixed?
Keep in mind #2528 and #2490 are separate issues and still valid, just concerned about the outline shader here (and any incremental improvements to resizing in general)

Shut. I think #2593 actually fixed #2490 rather than this one. Sorry for the mix up.
I did notice some problem with the outline shader (aka Sobel) but I have not investigated it.

While I'm trying to go through the code base I took a look at this issue. I can reliably reproduce the problem, but in my case, a black border appears. So if my window has a size of 1276 pixels and I rescale it the viewport for the whole world stays at 1276. The HUD though scales up.
After some debugging, I noticed that there is at least one SetViewportToSizeOfTask which still has the old window width saved. This leads to a glViewport call with the wrong width and to this problem.
I changed the glViewport call to always use the width of the window and this issue disappeared. I guess if we can find the reason why the width of this SetViewportToSizeOfTask is not updated on resize we can fix this.

This is what happens whenever you resize it in the launcher (which I always do) and then join the game and wait for a moment. Not sure how to replicate this 100%, but just resize it in the launcher and let it sit there for an hour. The difference between this one and the regular one, is the color. Goodluck!

FYI: "Launcher" == "Main Menu" don't forget :D
Not related to the TerasologyLauncher :-)
Other note relates to this including color unlike another symptom from resizing that results in black/grey/white everything

Closing as fixed by #2844!


updates to layout/texts

Changes:
Updated test-branch .gitignore to ignore _/sites
Fixed Title & Picture to match joshsoucie.com
Changed max-width for flex-container to 900px (bit skinnier)
Fixed conflicts with .masthead and h1
Added images to github folder

Approved.

Close updates.
Need to update joshsoucie.com to match


Use controller_path instead of controller_name for template path...

resolving. Makes partials in namespaces work.
Demonstrated in tests using Foospace namespace for BarController - putting include partial inside it tried to resolve bar/partial instead of foospace/bar/partial, like it should (and like Rails' partials work, always looking in current template path by default).

Coverage remained the same at 86.49% when pulling 75e15cea66f59be71c9d0c408d9d315a9f26eb9d on tomash:partials-respect-namespace into c794276fa88befd350a733644b8974a6980c1e6b on yoolk:master.

Coverage remained the same at 86.49% when pulling 75e15cea66f59be71c9d0c408d9d315a9f26eb9d on tomash:partials-respect-namespace into c794276fa88befd350a733644b8974a6980c1e6b on yoolk:master.

Coverage remained the same at 86.49% when pulling 75e15cea66f59be71c9d0c408d9d315a9f26eb9d on tomash:partials-respect-namespace into c794276fa88befd350a733644b8974a6980c1e6b on yoolk:master.

The failures in travis were both:
/home/travis/.rvm/gems/ruby-2.2.0/gems/activesupport-3.2.21/lib/active_support/dependencies.rb:251:in require': cannot load such file -- test/unit/assertions (LoadError)`

rebased and force-pushed, it seems the cause of failure has been already fixed in master. let's see if it builds!

Coverage remained the same at 86.49% when pulling 5d7bdad16da02b491e183887bad5a8f59ce9672a on tomash:partials-respect-namespace into c794276fa88befd350a733644b8974a6980c1e6b on yoolk:master.

still the same fail, apparently ruby 2.2 (or rather: its stdlib) does not like rails 3.2 and 4.0 :(

Thanks for this PR. It's okay for now to let rails 3.2 fail on ruby 2.2 and rails 4 on ruby 2.2 (it's because of rspec-rails, 3.1).


Migration to Rescript 11

Adds support for Rescript 11.

Rescript is now uncured by default, so is Tablecloth now
Bs-Jest is incompatible with Rescript 11
Replaced it with Rescript-Jest
Removed code that was required so Alcotest and Jest would work on the same set of tests

Overall with the current state of Rescript ecosystem it feels less and less productive trying to keep same API across different versions of the library. I'm thinking about creating a fork that will be tailored to JS developers, but we'll see.

Fantastic, thanks so much!
Can you address the errors in CI? https://app.circleci.com/pipelines/github/darklang/tablecloth-rescript/36/workflows/391e1ea0-055e-4587-822e-0cba2deca8ff
You probably need to make a small update to .circleci/config.yml

The lib was not building if the project version was <11, now it does (v9 and v10). Unfortunately, the tests in CI  for v9 had to be dropped, since rescript 9 could not build glennsl/rescript-jest as a dependency.


compileInsertArgs do not understand typed array

should be the same available as in https://github.com/Falseclock/dbd-php/commit/8dfb0bf5c9d2afcab26ab45ae4c4b9def5c183ec

enhanced


Split Harmonium into Harmonium and Harmonium-React

Or however we want to name them. We also have the Harmonium Phoenix elixir package as well.
Goals

Separate styles from react components. Allowing people to pull in styles without react components if needed
Keep Harmonium, Harmonium-React, and Harmonium-Phoenix versions in synced
Make/keep deployments easy

Going to close this. Since we don't really need it anymore


Updated Project list

Hello, thx for the electron-vue boilerplate. I wrote a ImapSync Client for Desktop.
I added my Project at the bottom of the project list.

Nice app @ridaamirini!


Find erb when importing glob

Including an erb in a globbed include would result in a File to import not found or unreadable error. This was happening because the ERB and Deprecated modules of the importer were both adding items with the same value to the extensions hash.  The sass gem's Filesystem importer does an invert on the extensions hash to apply the file's extension and was not resolving these correctly.  Changing the order of the imports resolves the issue.  This looks like the same problem reported in issue #185.

:+1:

Is there a viable way to include this change as a monkey patch in a Rails 4 app? It seems like re-ordering the includes as a monkey patch is difficult and/or might require refinements, but even that wasn't working for me. I tried adding an initializer, at the top was require 'sass/rails/importer', then the entire file was pasted below that with the include order changed, but it doesn't override the gem code. Likely because Ruby sees that the module is already included and doesn't include it again? I'm guessing there, but if I add a raise 'hello' in the monkey patch it is definitely getting executed. Any help would be greatly appreciated as it looks like this is only making it into v6 and beyond for some reason? Thanks!


windows libuv_1.x uv__handle_close crash

Version:
Platform:

I am in the test program find a crash.
Instructions:
1.platform windows x64 c++,libuv_1.x;
2.do_connect : new c++ class init connect and class object,now open a timer(3sec),if recv a full pack then close timer else acvtive destroy process.
3.destroy process : when the connect timer is activated,at this time close connect timer and open destroy timer(3 * 3 sec),at do_destroy callback,new a async event(async event work delete class object).
4.client : one thread new 1000 socket try connect server(do not send data),disconnect after 5sec,and again.
5.when 10000 times later crash.
6.crash position : tcp.c uv_tcp_endgame function uv__handle_close(277 line).
7.uv__handle_close :
(((QUEUE **) &(((((QUEUE **) &(((&(handle)->handle_queue))[1]))))[0]))) = ((QUEUE **) &(((&
(handle)->handle_queue))[0])); //-------crash line exception code : 0xC0000005 write memory error.
(((QUEUE **) &(((((QUEUE **) &(((&(handle)->handle_queue))[0]))))[1]))) = ((QUEUE **) &(((&
(handle)->handle_queue))[1]));
((uv_handle_t*) (handle))->loop->active_handles--;
(handle)->flags |= UV_HANDLE_CLOSED;
if ((handle)->close_cb)
{
(handle)->close_cb((uv_handle_t*) (handle));
}
8.now this handle flag == UV_TCP,is closeing.
hope for you reply!

Can you share the sample code for this on http://gist.github.com?

你能在http://gist.github.com上分享这个示例代码吗？

ok,Random code is log information and chinese text,use regular expression delete.
already shared! thanks!

1.find a new crash,at the uv_run--->uv__run_timers,handle->timer_cb crash,stack is broken.

Is there reason to believe this is a libuv bug rather than an application bug?

There's a lot more code here than I am willing to investigate, and uses a mess of non-standard formatting (plus RT for return, GG for goto, etc.) which make it hard to read. The full link to the code is https://gist.github.com/smbms/64175add965530a3f563db2c8eb1afdd
It does seem like it may have many use-after-free design issues too:
		uv_close(TO_HANDLE((*pv)[i]->GetConnectInstance()),NULL);
		DEL((*pv)[i]);

Thanks, I'll close this out then.
@smbms Please post to libuv/help if you still have questions.


feat: factory supports Node.js if a polyfill is present

Node.js doesn't provide an API for indexedDB. However, some
implementations do mimic it, by implementing a polyfill,
e.g. fake-indexeddb.
In such a case, it is nice to be able to use
indexed-db-futures. Hence this patch that adds a new
Global::node_global extern function, which returns a JsValue that
can be casted into a NodeGlobal. This latter provides an
indexed_db function, just like Window does.
All checks happen at runtime, like it was already the case, and it
will panic with an explicit error message.

Hey, man, I'm really sorry - your PR completely slipped under the radar. I'll get it merged & released in a moment

Thanks!


[AMENDMENT] ScriptSafe

Title
ScriptSafe (source: Security Tools ➜ Browser Extensions ➜ ScriptSafe)
Amendments
Fixed typo Ppera -> Opera
Amendment Data
name: ScriptSafe
url: https://www.andryou.com/scriptsafe
github: andryou/scriptsafe
icon: >-
  https://lh3.googleusercontent.com/c5co_NoLmEt48VC_yVp0JgKcgd83yiq_CdekGaOlBBfD5WII5mjxngERgikcQd4P56uoX9epiknU5ktXadPqj2EEVsE
description: >
  Allows you to block the execution of certain scripts. **Download**:

  [Chrome](https://chromewebstore.google.com/detail/scriptsafe/oiigbmnaadbkfbmpbfijlflahbdbdgdf)
  -

  [Opera](https://addons.opera.com/en/extensions/details/scriptsafe-2/)

Affiliation Disclosure
None
Would you like to submit a PR?
Yes!
Please tick the boxes

[X] You have filled out this form accurately, and to the best of your knowledge
[X] You have indicated whether or not you are associated with the project the amendment refers to
[X] A similar submission has not already been opened for this software / service
[X] You agree to our Code of Conduct
[X] You have read and followed the Contributing Guide

Fixed in https://github.com/Lissy93/awesome-privacy/pull/286. Thank you for the issue!


SPIFFS_tell returns wrong position after write

The following code:
spiffs_file fh = SPIFFS_open(fs, "test.txt", SPIFFS_O_CREAT | SPIFFS_O_APPEND | SPIFFS_O_WRONLY);
s32_t pos = SPIFFS_lseek(fs, fh, 0, SPIFFS_SEEK_END);
cout << pos << endl;
SPIFFS_write(fs, fh, "abcdefghij", 5);
SPIFFS_write(fs, fh, "abcdefghij", 8);
pos = SPIFFS_tell(fs, fh);
cout << pos << endl;
SPIFFS_write(fs, fh, "abcdefghij", 2);
pos = SPIFFS_tell(fs, fh);
cout << pos << endl;
SPIFFS_close(fs, fh);

produces the following output (if run 3 times):
pos1: 0
pos2: 8
pos3: 15

pos1: 15
pos2: 23
pos3: 30

pos1: 30
pos2: 38
pos3: 45

If I change line 1229 in spiffs_hydrogen.c from:
  res = fd->fdoffset;
to
  res = fd->offset;
I get the following (correct) output:
pos1: 0
pos2: 13
pos3: 15

pos1: 15
pos2: 28
pos3: 30

pos1: 30
pos2: 43
pos3: 45

Any help or hints about the correct way to fix this would be much appreciated!
Note: we are shipping product with the file system configuration and everything has been working well. I'm confident that the settings are good.

Flushing didn't help. It looks to me like SPIFFS_Tell() already flushes before getting the offset:
s32_t SPIFFS_tell(spiffs *fs, spiffs_file fh) {
  SPIFFS_API_DBG("%s "_SPIPRIfd "\n", __func__, fh);
  s32_t res;
  SPIFFS_API_CHECK_CFG(fs);
  SPIFFS_API_CHECK_MOUNT(fs);
  SPIFFS_LOCK(fs);

  fh = SPIFFS_FH_UNOFFS(fs, fh);

  spiffs_fd *fd;
  res = spiffs_fd_get(fs, fh, &fd);
  SPIFFS_API_CHECK_RES_UNLOCK(fs, res);

#if SPIFFS_CACHE_WR
  res = spiffs_fflush_cache(fs, fh);
  SPIFFS_API_CHECK_RES_UNLOCK(fs, res);
#endif

  res = fd->fdoffset;
//  res = fd->offset;  <<-- this seems to work

  SPIFFS_UNLOCK(fs);
  return res;
}

Thanks @pellepl ! Works great.


Is there a way to avoid relogging the user on reload?

I notice the connectionLogin and connectionLogout events will count every refresh or visit to another route as a new login. Is there a simple way to bypass the update and only count when the user logs in/out?

That certainly won't happen if you use something like Iron Router which will change the route without reloading the app. But since Meteor doesn't support connection resuming at the moment, we can't tell the difference between a user leaving and returning vs. just reloading.

Is it possible to stop this behavior in FlowRouter?


Extend Performance.now polyfill range for Android Browser.

Fixes #1020

Is it released on polyfill.io?


Consider groups of translator options based on use case for ELM

Currently, translatorOptionsMatch is built to include any change in CQL behavior and/or ELM output. Consider more fine-grained controls to support different use cases for ELM usage:
Master option:
EnableAnnotations
Language Features:
DisableListTraversal
DisableListDemotion
DisableListPromotion
EnableIntervalDemotion
EnableIntervalPromotion
DisableMethodInvocation
RequireFromKeyword
Annotated Helpers for Interpreters:
EnableResultTypes - function overloads
SignatureLevel - function overloads
Dependency Loader Control:
EnableCqlOnly - Doesn't load pre-compiled ELM (JSON or XML) dependencies
CompatibilityLevel - Compatibility level for the translator, valid values are 1.3, 1.4, and 1.5
Debbuging Information for Language Servers.
EnableLocators - Find source of errors
ErrorLevel - Error, Warnings, Info, etc.
EnableDetailedErrors -
EnableResultTypes - Code completion
I think the most difficult one is CompatibilityLevel, which is currently used in the dependency library loader but in a way that only loads libraries that declare exactly the same compatibility level value the loader is expecting as opposed to just using the option in the compiler to check if the library is compatible with that version (removing the need to compare the annotations themselves)

https://github.com/cqframework/clinical-reasoning/pull/286


Added Hello World in Delphi

Created Delphi.dpr

Please rename the filename to be all lowercase and add an ! After the word "World"

Updated the Changes


Closes #2693 - Adds additional colors for cables & roles

Fixes: #2693
Adds to the color picker:

White
Rose
Aqua

Additionally matches on the background color white on a label and reverses the font color to black.

@DanSheps The CSS hack is no longer needed; see f4a22e5. If you remove that this should be good to merge.

@jeremystretch Thanks, will do it this week.


Search in the code, but, only find code that is uncommented.

I'm sure a lot of people can use this. Basically: anything that is commented out: is automatically excluded from the search.
Regex can do it but its code can get much longer than the search term itself.

#11688 covers this. The title given to that issue by its author mentions "Find" rather than "Search", and it has been labelled editor-find rather than search, but the description and all subsequent discussion is about VS Code's Search view and not the Find-in-file widget.
@roblourens please consider updating that issue's title and labels.
/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


ADBDEV-4377: gpbackup does not backup statistics for leaf partitions with leaf-partition-data option

gpbackup does not backup statistics for leaf partitions with leaf-partition-data option
This patch solves the problem by backuping statistics for all specified
tables, including leaf partitions, and not just root partitions.
The test for this patch is based on the test for the 85384fadf774560b87df4d30792644b87069583e.
And, as stated in that patch, it removes the unnecessary
condition when comparing the number of statistics.

You should change branch ADBDEV-4376 to master.

Done

I think, you should rewrite the head of commit message. Head should answer the question: "what had been done?". For example: "Fix something at somewhere".

you should rewrite the head of commit message

I changed the name and content of the PR. Will this work for a commit? If so, can I force push changes?

I changed the name and content of the PR. Will this work for a commit? If so, can I force push changes?

You can add empty commit with new commit message without force-push.


chore: update CODEOWNERS

Regrettably @magedhennawy has left our core team 😢
Looking forward to your first PR as an external contributor!

As much as those emails got to me.............. this makes me sad 😢

Travis is rate limited or something right now, but since this isn't a code change I'll merge it in anyway


Table row gets broken when using back-quoted backslash()

VSCode Version:
1.8.0-insider
0b847998d914ff84fd9c13b2972cc8803745c24d
2016-11-25T09:18:45.881Z
1.4.6
53.0.2785.143
Node 6.5.0

OS Version: Windows 10 Insider Preview 14971

You can see the 4-th row is not correctly handled by VSCode's markdown preview:
#|a|b
--|--|--
1|`a` | `b`
2|`\a` | `b`
3|`\` | b
4|`\` | `a`
5|`\ ` | `a`

#
a
b

1
a
b

2
\a
b

3
\
b

4
\
a

5
\
a

The 5-th line just appends a white space after \ and it prevents from breaking the table row.

Thanks. This looks like an issue in the markdown-it library we use for the preview.
I've opened an issue against them to track this: https://github.com/markdown-it/markdown-it/issues/303


Support more data types

I understand the waterline way which is to ensure all db adapters work.  Why don't we add support for the types a db uses, for example on Postgre uuid and decimal, and then for other adapters that dont use those types we just set a type it does use.  This way we can utilize the correct data types for a given database.
Example:
Mongo doesn't have a uuid type, so that adapter might return a string
case "uuid":
  return "string";

whereas on the postgre it would use that type.  This would keep things working the way waterline is designed and would allow us to use the correct data types.  I'm willing to add these types and update the existing adapters.
Update: Or we can just have a default catch all that returns a string if none of the case types matched.

@GaryLCoxJr, your request is very similar to #819 (PR #832) so I'm merging both issues in #819. Please continue the discussion there. Thanks.


openfhe-python for models

hi, how can I encrypt model.state_dict() using openfhe-python?
BR

Can you share your code or elaborate further? I don't think there are model capabilities built directly into openfhe-python.

@macknight
Can you follow up on if this issue was solved? If not I'm going to close it


How to custom the newline height?

Does Markwon support to custome the newline height?
for example, we would like change the H tag newline height be 8dp, how to config it? thanks.

Hello @zhuosun-rally !
It is possible of cause. There is no configurable option out-of-box, but you can control how the new-line character is displayed.
final int dip8 = (int) (context.getResources().getDisplayMetrics().density * 8 + .5F);

final Markwon markwon = Markwon.builder(context)
        .usePlugin(new AbstractMarkwonPlugin() {
            @Override
            public void configureVisitor(@NonNull MarkwonVisitor.Builder builder) {
                builder.on(Heading.class, new MarkwonVisitor.NodeVisitor<Heading>() {
                    @Override
                    public void visit(@NonNull MarkwonVisitor visitor, @NonNull Heading heading) {
                        // this is how core plugin visits Heading nodes

                        visitor.ensureNewLine();

                        final int length = visitor.length();
                        visitor.visitChildren(heading);

                        CoreProps.HEADING_LEVEL.set(visitor.renderProps(), heading.getLevel());

                        visitor.setSpansForNodeOptional(heading, length);

                        if (visitor.hasNext(heading)) {
                            // save current index
                            final int start = visitor.length();
                            visitor.ensureNewLine();
                            visitor.forceNewLine();
                            // apply AbsoluteSizeSpan to new-line character
                            visitor.setSpans(start, new AbsoluteSizeSpan(dip8));
                        }
                    }
                });
            }
        })
        .build();

Please note that this solution uses AbsoluteSizeSpan which might result in the line-height not exactly of 8dip. There is the LineHeightSpan interface in Android, but it does not provide implementation, and by looking at stackoverflow answer it seems like a bit too much. Anyway, you can try this also

Thanks @noties, it works for me,
if you can put them to theme config or somewhere will much convenience, like paragraph space height and heading space height ^^,
I will close the ticket, thank you.

@zhuosun-rally I will have to think of how to do this customization. Meanwhile if you have any thoughts, they are welcome


Add brandingmaterials

Added 2 folders 'neg' and 'pos', containing the logo in negative (white on black bg) and positive (black on white bg) version.

Wrong place to put these

Wrong place to put these


Impossible to edit articles under section Interface Views on localize version

I can't see "Edit" link, so opportunity to edit a translation is missing.
For example here : https://docs.microsoft.com/ru-ru/xamarin/xamarin-forms/user-interface/
The same as, probably, for the whole section.

@JasonCard Is this something you can help with?

Hi @FoggyFinder,
Thank you for your feedback and suggestion.
The article https://docs.microsoft.com/ru-ru/xamarin/xamarin-forms/user-interface/ is machine translated.
Currently, we are discouraging users from sending PRs on all machine translated content and have thus turned off the "Edit" button for this content.
We use our human translated content to train the machine translation engines, so over time, this will improve the quality of all translations.
We cannot ensure that in the future we will process machine-translated content PRs, so it would be best to focus your efforts on the human translated topics, which will have much greater impact. We are closing this PR now.
Many thanks for your support.
Kind regards,
Microsoft DOCS Global Experience Team
@davidbritch Could you please go ahead to close this issue as I don't have access to close it.


fix: improve tests performance with custom harness

Cleaned up the code, and added proper multi-threading.
Locally this is around 50% faster(from ~30s to ~15s). And provides better overall control for running tests, hopefully multi-threading doesn't cause bugs here, from my testing I haven't had any issues with it locally.

Do you know what is going on here? Would be happy to merge when the tests pass.

I have not being able to reproduce the issue on any windows systems or servers, so I think this is very specific to the windows server for github actions. Will separately debug it on my own repo within the action itself, until then I am moving it to draft. Might generate some noisy commits here as well so disabled CI here.
for the moment.


Internationalization Support

Add support for UI elements to appear in different languages.

We should explore this for upcoming deployment with english & spanish content in the credential


Cover fails to get current_cover_position for device 'rts:GenericRTSComponent'

This device is an RTS micro module to command covers, but as it is rts it doesn't have state return, hence the failure...
This device should be mapped as a switch probably...

uiClass is Generic

I believe this should still be mapped to a cover, since it is a cover device. We already set assumed_state to True for RTS devices. It should be possible to have a cover without a position.
How do other components do this?

https://github.com/iMicknl/ha-tahoma/blob/34b6832f554d02f8455814fbc8e51aa92388ab02/custom_components/tahoma/cover.py#L103
Maybe recent changes have caused this? It should return None when there is no position available?

yes, it's quite recent, the addition of helper functions

Yes indeed, we forgot to handle the None case.


Unable to change postgres port

If I change the postgres_port in config.yaml, the service no longer boots. I believe it's because the port is mapped 3333:3333 (assuming 3333 is the new port), but doesn't change the internal postgres port. If I'm looking at this correctly, it should be mapping the custom port to the default port. Ie 3333:5432.

@defrex good catch! fixed it on a PR I already had for warning when a new CLI version is released to npm. In this initial phase, I presume lots of updates will be pushed on a regular basis.


Fix flowchart syntax

Description of the bug
The flowchart is printing '\n'. This should be fixed.
Command used and terminal output
No response
Relevant files
No response
System information
plant-food-research-open/assemblyqc version 2.1

https://github.com/Plant-Food-Research-Open/assemblyqc/releases/tag/2.1.1


Dell U2414H Monitor not supported (M1 Beta)

Checklist

[ x] I have searched for existing issues
[ x] I have looked through the wiki
[ x] I have updated MonitorControl to the latest version

Describe the bug
A clear and concise description of what the bug is.
The above monitor is recognized . The App cant control him .
Expected behavior
A clear and concise description of what you expected to happen.
Adjust the Monitor .
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.
Environment Information (please complete the following information):

macOS version: 11.4
Mac model: MacBook Air M1
MonitorControl version: Beta 3
Monitor(s): Dell U2414H
Apple Silicon/M1 (yes or no): yes

Can you try beta 7 and report if it's working better?

Hmm. I think the Dell 2414H is connected via a DisplayLink dock as M1 MBAs don't support two external displays and the display ID will never be 3 for native devices on M1.
Unfortunatelly DisplayLink does not support DDC on Mac.

@F3000 I close this now. Please let me know if you have anything to add though! Thanks!


Fix text color for path of manual import in Radarr

Can you upload a screenshot of where this is?

Manual import. You need to try changing/searching for the movie, and the path appears at the bottom


OrderLine says value 0 is null

Getting a error when doing a Post checkout/v3/orders,
that one of my orderLines that has the value 0 on tax_rate and total_tax_amount
Bad value: order_lines[3].tax_rate (may not be null), order_lines[3].total_tax_amount (may not be null)
It did work fine to send 0 as tax_rate and total_tax_amount when sending with a request with all data as json.
Is this intended?

A workaround to this is to create the Klarna Client with a custom JSON serializer as shown here https://github.com/klarna/kco_rest_dotnet/blob/v3.0/Klarna.Rest/Klarna.Rest.Core.Examples/CustomJsonSerializer.cs.
And in the custom serializer set:  DefaultValueHandling = DefaultValueHandling.Include

I merged change you offered. It's quite dangerous change, so it will go to a new 3.1.x version soon.
Thank you for contributing.

When can we expect this to be implemented AKA v. 3.1 released?


Overflow menu does not position correctly if browser is zoomed in

Detailed description
The overflow menu checks assumes top, left, width, height and border-top-width will all be whole pixel values for a target pseudo element.
.#{$prefix}--overflow-menu-options:before
This is an invalid assumption when zooming in and/or out the browser.
This
[name]: Number((/^([\d-]+)px$/.exec(arrowStyle.getPropertyValue(name)) || [])[1]),
should read
[name]: Number((/^([\d-.]+)px$/.exec(arrowStyle.getPropertyValue(name)) || [])[1]),

Is this issue related to a specific component?
Overflow menu although a similar test exists in tooltip.

What did you expect to happen? What happened instead? What would you like to see changed?

Overflow menu positioned correctly

What browser are you working in?

Chrome 67

What version of the Carbon Design System are you using?
9.1.20

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

Steps to reproduce the issue
NOTE: Occurs here https://codepen.io/team/carbon/full/MOEwjp/

zoom in once
open overflow menu
Note the offset is not calculated correctly.

Additional information

Thanks alot for catching @lee-chase! Appreciated if you can submit a quick fix as you proposed.


Trigger signature_help on comma

Fixes an issue where some tools are not properly triggered for signature help.
See https://github.com/ray-x/lsp_signature.nvim/issues/38

Thank you for this! :heart:


[BUG]: Request Denied even after inputing an API key

Bug Description
I've followed the directions, installed the module, copied the API key and pasted it within the settings for the module, but I keep getting the request denied message
Steps To Reproduce
Install module
Generate API key
Paste API key in module settings
Go to character tab
Click create NPC
Generate

Foundry Version
Foundry 11.315
Additional Information
No response

send me the console log error screenshoot, here or on discord, whichever is more convenient for you.

Seeing the same issue on Foundry v. 11.313 and module v. 1.5.4:
Access to fetch at 'https://api.openai.com/v1/chat/completions' from origin 'https://redacted' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

POST https://api.openai.com/v1/chat/completions net::ERR_FAILED 503 (Service Unavailable)

TypeError: Failed to fetch
    at npcGenGPTLib.callAI (lib.js:28:36)
    at npcGenGPTGenerateNPC.initGeneration (generateNPC.js:61:49)
    at HTMLButtonElement.dispatch (jquery.min.js:2:43184)
    at y.handle (jquery.min.js:2:41168)

Seeing the same issue on Foundry v. 11.313 and module v. 1.5.4:

@jcomplex, did the issue occur only once, or does it happen every time the module is used? are you on the forge?

I am getting the same issue here is my error log
NPC Generator (GPT) | Sending Request
api.openai.com/v1/chat/completions:1     Failed to load resource: the server responded with a status of 500 ()
foundry.js:68326 NPC Generator (GPT) | Request Denied! Wait a few seconds and try again. (See console (F12) for more details)
fetch @ foundry.js:68326
lib.js:40 Error: 500 | Message: Internal server error
at npcGenGPTLib.callAI (lib.js:34:23)
at async npcGenGPTGenerateNPC.initGeneration (generateNPC.js:61:30)

It's a OpenAI issue. The API status page reports:
Elevated Errors on API and ChatGPT
Identified - The issue has been identified and a fix is being implemented.
Investigating - We're experiencing an elevated level of API errors and are currently looking into the issue.
https://status.openai.com/

I'm not sure their reported issue is what's causing these denies.  May be another that they're unaware of, but I'm still experiencing the issue and they reported a fix on 11/23.

I am having this same issue, it seems to be code  429 error is what I think I need to resolve, (See Screen Shot)

I'm no expert, so I'm just spamming my thoughts.

@Ruinwolf as you can see from the error it reports that you have no credit on your openAI account, that's why it refuses your call. Note, as I mention in the readme file, that OpenAI APIs are not free to use although they are really cost-efficient.


Include message text in the URL Preview Widget

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #android-bugs.

Would issue #797 be closed then?
Here are some example images I sent in the discord

Done


Setup custom error class

We need a PyspextoolError when the built-in error types don't fit.

https://www.programiz.com/python-programming/user-defined-exception

This is what we have in SIMPLE:
class SimpleError(Exception):
    pass

More reference material:

https://www.geeksforgeeks.org/user-defined-exceptions-python-examples/
https://martinxpn.medium.com/custom-exceptions-in-python-creating-custom-exceptions-59-100-days-of-python-4f26de8e851d

One thing I'm not sure about is where this should get defined.  It should be in a module which is imported by everything.


LOD detail levels wrong

It looks like the LODing is much stronger since version 3.9 it is at least double that agressive, maybe even 4 times.
I had setup all my models so the LOD popping is hardly visible far in the distance, now I constantly see models popping right infront of me, if I set the settings to lowest there is even whole forests disappearing  right infront of me.
So there must be something seriously wrong with that. Maybe the pixel distance to determine how big the objects are is wrong or the detail adjust rate from $pref::TS::detailAdjust

As I recall it, it was more about making the LOD transitions consistent and reliable in when they occured, as the old logic wasn't quite right, especially at certain aspect changes.
So it could very well be that the default assets need to have updates to their LOD numbers, the transition kickoff should be more correct now than before.
If you've got a video of any behavior you think is erroneous, that could help figure out if there's a bug, or just that assets need updating.

Consistent in what way?
Saying that everyone has to update all their models now is a bit bold, especially not knowing what has been changed exactly and that it may be wrong as well, so one would update like 60 models by hand like I have all in Vain then.
I know that it is probably wrong, since I have a nulldetail level at last or am using the "$pref::smallestVisiblePixelSize" for it and they obviously pop out much sooner, with low mesh settings disappearing right in front of your face.
I tested an object with normal settings which is smallestVisiblePixel = 4 in my case and the shape disappears after it is lower than 8 pixel in height and lower than 16-20 pixel diagonal, so it is approximately 4-5 times stronger than it should be.
So am I supposed to use 0.5, 0.25 (which is impossible since 1 pixel is the smallest) values now for smallestVisiblePixel to make up for that miscalculation?
I tested it by zooming out seeing where it disappears, making a screenshot and then counting the pixels the object is big right before it disappears, then you get the size at which it disappears.

I made some tests with screenshots showing the difference between 3.8 and 3.9:
http://duion.com/files/temp/3.8lod.jpg
http://duion.com/files/temp/3.9lod.jpg
As you can see 3.9 LODs far earlier then the 3.8
The rock has detail levels 200, 80 and 4 and the 3.8 version properly LODs at 200 pixels from the 200 detail level to the 80 detail level while the 3.9 version seems to LOD from the 200 detail level at when it is 80 pixel big, so maybe someone confused the values there or skipped a LOD level.

I did measure the size of the model in the screenshots when it switched from the 200 pixel detail level to the 80 pixel detail level and in the 3.8 version it is 40 pixel and in the 3.9 version it is 130 pixel, so both are wrong, it should be 200 pixels where it switches away from the 200 pixel detail level to the 80 pixel detail level.

More values with detailAdjust 1 and with the default soldier:
3.8: It switches from the 300 detail level to 200 detail level at approximately 150 pixels diagonale
3.9: It switches from the 300 detail level to 200 detail level at approximately 450 pixels diagonale

In tsShapeInstance.cpp around line 597 the LOD is calculated.
const F32 pixelScale = (state->getViewport().extent.x / state->getViewport().extent.y);
if you change this line to
const F32 pixelScale = (state->getViewport().extent.x / 300.0f);
you will have the old LODding.

Good to know, currently I use:
const F32 pixelScale = (state->getViewport().extent.x / state->getViewport().extent.y)*2;
Looks quite crazy to just add a magic number, but from my tests it is pretty accurate in most cases.

#1750 merged in, which uses the *2 quickfix.
For 4.0, we'll move to a more proper screen-space value for controlling it, but that's a separate thing. For now, this brings the behavior back in line with 3.8.


Flag to confirm results before reporting

Hello,
I think it would be great if there was a flag to confirm items a second time before reporting it as a find. I've noticed some hosts throw out different responses here and there, which look like a valid finding. This could be filtered out by simply requesting that URL a second time to see if it still looks valid.
Thanks!

Isn’t this already addressed by being able to add regex checkers to exclude results you find to be false flags?

How would I know what to use as my fr flag if the errors happen sporadically? Plus, that would require me to stop my scan, add the regex, then restart the scan. I run ffuf as part of a larger script, so that's another reason the fr flag wouldn't be an option for me.

So what do you mean by requesting the url a second time to “see if it looks valid”. Urls return a pretty non-fluctuating set of known http codes. If a 200 for you isn’t the type of valid you’re looking for, how would requesting it twice fix this? Appropriate ffuf workflows generally contain some kind of specific regex tailoring to the specific target you’re focused on.

I'll give an example. Let's say you run ffuf -w mywordlist.txt -u https://example.com/FUZZ -ac -mc all. Everything is going great, except 20k requests in you get a few 504s. Ffuf reports those as valid results because there's nothing indicating that these aren't valid results. All ffuf knows is that it got a different status code for a few paths.
The current solution for this would be to ctrl+c, add -fc 504, then re-run the scan.
If ffuf were able to confirm results by requesting them one more time to ensure it wasn't a fluke, that would be helpful for someone like myself that uses ffuf for automation. I manually visit these URLs to ensure they were flukes, then remove them from the results by hand.

504’s aren’t flukes. They are real responses indicating something. If you know you won’t want 504’s, set your automation script to exclude 504’s.

In the example I'm giving, the 504 has nothing to do with the path given. Yes, it indicates that the server had trouble fulfilling the request, but that doesn't mean the result is valid. The payload used has nothing to do with the result received, therefore it would be nice if it was re-validated.
It's sounding like this is an issue specific to me, so I'm fine forking the projecting and adding the flag for myself.


Update all node icons to SVG

When listing icons provided by a module, if there is a png and svg
with the same name, only the svg will be listed.
If a node asks for a png icon which is not known, but there is a
corresponding svg, that will be used instead.

This is a quick sample of how the improved icons alongside their png originals. Not all icons have been updated (yet).

Might be worth looking at Terence Eden's https://github.com/edent/SuperTinyIcons for the Twitter icon (and any other social media icons)

Coverage decreased (-0.04%) to 76.691% when pulling 0b204de5a90f81aa2aee804d4e28a53ea46c0598 on svg-icons into 92cb57eb7b949f5e043f4c5c64dd8877118e5174 on dev.


debug/macho: add missing constants

Current code doesn't have flags such like MH_DYLDLINK, MH_PIE, ... etc.
Which is needed for tests.

Reopening because the CL didn't add all the file flags, just a few of them.


Consolidated the C# nullable value types articles

Fixes #14726
Supported by dotnet/samples#1694
Contributes to #13421

@BillWagner I've addressed your feedback with one item waiting for your response.

This looks great @pkulikov
I'll :shipit: now.


DOCS (guide): change Generating Components, Directives, Pipes and Services section

Change in a update version of README to create a new PR (as suggest by Brocco)

@filipesilva done 😃


Implement communication between frontend and backend using FastAPI

✨ Desired Solution
Develop a backend using FastAPI to handle API requests from the frontend, process the requests, and send responses back to the frontend.
📚 Technical Details
- Use FastAPI to create the backend server.
- Define API endpoints for communication with the frontend.
💡 Examples and References

Reference FastAPI documentation
Look at existing examples of FastAPI integration with frontend frameworks.

🔗 Subtasks

[x] Set up the FastAPI server.
[x] Define API endpoints for communication with the frontend.
[x] Implement request handling and response sending in the backend.

Checkout #27, it contains tasks related to starting/stopping simulation and transmitting its data

Since we now have the basic backend setup I'll close this issue and we continue on #27. We'll get back to creating concrete APIs in the future


Correctly lookup assets when using a relative build directory

Fixes https://github.com/facebook/create-react-app/issues/5162

e2e would be nice

Example broken e2e:
joes-mbp:create-react-app joe$ ./node_modules/.bin/jest --config fixtures/smoke/jest.config.js fixtures/smoke/relative-paths/
 FAIL  fixtures/smoke/relative-paths/index.test.js (22.318s)
  relative paths
    ✕ builds in production (21559ms)
    ○ skipped 1 test

  ● relative paths › builds in production

    expect(received).toBe(expected) // Object.is equality

    Expected: "/private/var/folders/c3/vytj6_h56b77f_g72smntm3m0000gn/T/72a73262107e4b983270278cf537ee30/build/static/media/logo.83ec6b6a.svg"
    Received: "/private/var/folders/c3/vytj6_h56b77f_g72smntm3m0000gn/T/72a73262107e4b983270278cf537ee30/build/static/css/static/media/logo.83ec6b6a.svg"

      31 |       .exec(fs.readFileSync(cssFile, 'utf8'))
      32 |       .pop();
    > 33 |     expect(path.resolve(path.join(path.dirname(cssFile), desiredPath))).toBe(
         |                                                                         ^
      34 |       path.resolve(svgFile)
      35 |     );
      36 |   });

      at Object.toBe (relative-paths/index.test.js:33:73)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 skipped, 2 total
Snapshots:   0 total
Time:        22.367s, estimated 23s
Ran all test suites matching /fixtures\/smoke\/relative-paths\//i.
joes-mbp:create-react-app joe$


Cannot add documents (only in a dev branch): unsupported snappy compression

⚠ This bug is only on this branch, but this branch is used by the @meilisearch/cloud-team in their dev SaaS via the v0.26.1-cloud-team tag
It's impossible to add documents, getting an internal error.
Steps:

Add documents

curl \
  -X POST 'http://<IP_ADDRESS>:7700/indexes/movies/documents' \
  -H 'Content-Type: application/json' \
  --data-binary '[{"id": 1}]'

Check the task status and get

{
      "details": {
        "indexedDocuments": 0,
        "receivedDocuments": 1
      },
      "duration": "PT0.001297S",
      "enqueuedAt": "2022-03-19T15:08:41.940767Z",
      "error": {
        "code": "internal",
        "link": "https://docs.meilisearch.com/errors#internal",
        "message": "unsupported snappy compression",
        "type": "internal"
      },
      "finishedAt": "2022-03-19T15:08:41.942588Z",
      "indexUid": "movies",
      "startedAt": "2022-03-19T15:08:41.941291Z",
      "status": "failed",
      "type": "documentAddition",
      "uid": 2
    }

Thank you @curquiza for this issue, it seems like the issue comes from this line that correspond to the arguments used by Meilisearch and that were exposed in the latest v0.26.1-cloud-team release.
https://github.com/meilisearch/meilisearch/blob/b2bbd13c27c5719e5aea53fc60815975b5aa422b/meilisearch-lib/src/options.rs#L35
We must remove the default value setting. This way the Default implementation is used and CompressionType::None is used by the engine (which is expected).
By the way this argument can be removed without consequences. To be checked.

Fixed on #1928 💪


#340 use transaction for manual capturing

Fixes: https://github.com/commercetools/commercetools-adyen-integration/issues/340

please update :

payment type
json validations
constants.

https://github.com/commercetools/commercetools-adyen-integration/pull/341/commits/846974cdb794c986884ba4880c2b1ef1c60c0906


Add payment functionality

I would like to add payment functionality once the user is registered. Please assign me this issue under gssoc'24.
I am contributor at gssoc'24
please assign this issue to me @abhay-raj19

I am interested in working on this issue and adding payment functionality to the project. I will implement a secure payment gateway and ensure that the payment process is smooth and user-friendly. Please assign this issue to me under GSSOC'24 label.

That's also looks Good Issue , and will wait for the PR !!

I am interested in working on this issue and adding payment functionality to the project. I will implement a secure payment gateway and ensure that the payment process is smooth and user-friendly. Please assign this issue to me under GSSOC'24 label.

Raise up the PR for same if you prefer some new payment gateway like stripe or some other .
will label it with GsSoc'24

Thank you for assigning this issue to me.
On Fri, May 10, 2024 at 9:57 PM Abhay Rajpoot @.***>
wrote:

Assigned #102 https://github.com/abhay-raj19/FitBody/issues/102 to
@InshiyaRavat https://github.com/InshiyaRavat.
—
Reply to this email directly, view it on GitHub
https://github.com/abhay-raj19/FitBody/issues/102#event-12771736715, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXAO36MXN2R4SJIJHOYW4O3ZBTYNRAVCNFSM6AAAAABHQ3MA22VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSG43TCNZTGY3TCNI
.
You are receiving this because you were assigned.Message ID:
@.***>

--
REGARDS,
INSHIYA RAVAT.

@abhay-raj19  Okay Assign this to me I will continue with adding stripe payment gateway

okay, assign it to. me will work on this  @abhay-raj19


enabling encoding; refactor to use constants

fixes #308. it would be great to get this in v1.1.
aside: when is v1.1 due out?

Thanks for this PR.  I can definitely get this into 1.1.  I will try to do it this weekend.


Support more certificate loading scenarios (#69)

#69

@cesarbs,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

📢

We shouldn't throw if we look for certificates in store and don't find any. Instead we should throw where those certificates are being used (ex identity/kestrel).

@cesarbs, @DamianEdwards thinks we should multi-target netstandard1.3 and netcoreapp2.0 so that we keep supporting desktop CLR apps.


Multiple SQL injection vulnerabilities in the Doctrine\DBAL

Any reason the 1.x isn't included?
In security-advisories/doctrine/dbal/2011-09-25.yaml
title:     SQL injection possibility
link:      https://www.doctrine-project.org/blog/dbal-security-2011-1.html
cve:       ~
branches:
2.0.x:
time:     2011-08-29 22:36:11
versions: ['>=2.0.0', '<2.0.8']
2.1.x:
time:     2011-08-29 22:36:11
versions: ['>=2.1.0', '<2.1.2']
reference: composer://doctrine/dbal
But in NVD the description says this:
Multiple SQL injection vulnerabilities in the Doctrine\DBAL\Platforms\AbstractPlatform::modifyLimitQuery function in Doctrine 1.x before 1.2.4 and 2.x before 2.0.3 allow remote attackers to execute arbitrary SQL commands via the (1) limit or (2) offset field.

There is no DBAL 1.x


Compile only the *.c files used by your display driver

Currently, all the source files located on the lvgl_tft directory are being included for compilation, even those not used by your driver. Sometimes this triggers errors on source files not necessary for your driver.
Consider including only the source files used by the currently enabled driver.

Fixed in 8ba9d9d1662400b0626fa7f4766f34f234c30a08


GDScript: Add static analysis error reporting in GDScriptCache::get_full_script()

Currently, GDScriptCache::get_full_script() emits errors during file opening and parsing, but not during static analysis or compilation. This PR changes it so analysis/compilation errors are emitted.
It is essentially a continuation of what #76954 did for GDScriptCache::get_shallow_script().
:warning: this will cause scripts to fail earlier, possibly causing more cascading errors, which was also a worry I had with #76954 :warning:

Thanks!


Feature/django conventions

This one includes previous PR (https://github.com/makimo/style-guide/pull/1). Is it intentional?

It was - just for my own convenience when writing the .md file.


Added support for these boards from Embedded Artists:

LPC4088 Display Module (LPC4088DM)
LPC4088 QuickStart Board (LPC4088QSB)

Please accept the contributors agreement:
https://developer.mbed.org/contributor_agreement/

@mbed-bot: TEST